Crash by cyclic use of Boolean Modifier #32866

New Issue

Shinsuke Irie · 2012-10-13T04:50:06+02:00

Shinsuke Irie commented

2012-10-13 04:50:06 +02:00

%%%Steps to reproduce:

Create 2 cubes (cube1, cube2)
Add boolean modifier to cube1 and set Object=cube2
Add boolean modifier to cube2 and set Object=cube1

Then Blender crashes immediately outputting the following message:

Dependency cycle detected:

cube1 depends on cube2 through Boolean Modifier.
cube2 depends on cube1 through Boolean Modifier.

Segmentation fault (core dumped)
$

Such use of the modifier makes no sense, but we might do that accidentally and lose our data unexpectedly.

r51289
Ubuntu 12.10 amd64
%%%

%%%Steps to reproduce: 1. Create 2 cubes (cube1, cube2) 2. Add boolean modifier to cube1 and set Object=cube2 3. Add boolean modifier to cube2 and set Object=cube1 Then Blender crashes immediately outputting the following message: Dependency cycle detected: ``` cube1 depends on cube2 through Boolean Modifier. cube2 depends on cube1 through Boolean Modifier. ``` Segmentation fault (core dumped) $ Such use of the modifier makes no sense, but we might do that accidentally and lose our data unexpectedly. r51289 Ubuntu 12.10 amd64 %%%

Shinsuke Irie commented

2012-10-13 04:50:06 +02:00

Changed status to: 'Open'

Bastien Montagne commented

2012-10-13 11:19:35 +02:00

%%%We just run into an infinite loop, as boolean modifier expects the final mesh (i.e. with all modifiers applied) of its second operand…

Not sure though how to nicely fix that, perhaps returning to 2.63 code in applyModifer when getting second operand's dm (it indeed fixes the crash)? Or placing a sort of "watchdog" to prevent recursion?

Will leave this one to sergey! :)%%%

%%%We just run into an infinite loop, as boolean modifier expects the final mesh (i.e. with all modifiers applied) of its second operand… Not sure though how to nicely fix that, perhaps returning to 2.63 code in applyModifer when getting second operand's dm (it indeed fixes the crash)? Or placing a sort of "watchdog" to prevent recursion? Will leave this one to sergey! :)%%%

Sergey Sharybin commented

2012-10-15 14:04:05 +02:00

%%%That was a change from Campbell to support some special cases when objects from different layers are used.

Campbell, this indeed should be solved in a different way. Could you look into this?

Side note: the same could happen with array modifier when recursion appears on Start/End cap objects.%%%

%%%That was a change from Campbell to support some special cases when objects from different layers are used. Campbell, this indeed should be solved in a different way. Could you look into this? Side note: the same could happen with array modifier when recursion appears on Start/End cap objects.%%%

Brecht Van Lommel commented

2012-11-10 04:24:10 +01:00

%%%At some point I changed all modifiers to use only derivedFinal and just fail if it is not available. This reveals issues in our dependency graph of course, which may not be possible to fix without a major refactor.

If this is an important case to have working now, it could do some manual check for recursion, like a flag in the modifier that is set/cleared before/after running.%%%

%%%At some point I changed all modifiers to use only derivedFinal and just fail if it is not available. This reveals issues in our dependency graph of course, which may not be possible to fix without a major refactor. If this is an important case to have working now, it could do some manual check for recursion, like a flag in the modifier that is set/cleared before/after running.%%%

Moolah Nasreddin commented

2012-11-10 14:24:01 +01:00

%%%IMO, Blender lacks of "solid" working modifiers for creating full range of modern particle and mesh FX. So by now I suppose it's really better to have some hacks (before that major refactor) for several modifiers than NOT to have them. Boolean is a very important modifier for creating complex effects without tricky workarounds.%%%

Campbell Barton commented

2013-01-23 06:45:33 +01:00

%%%Committed workaround r54017, but ideal but avoids crash.%%%

Campbell Barton commented

2013-01-23 06:45:34 +01:00

Changed status from 'Open' to: 'Resolved'

Campbell Barton closed this issue

2013-01-23 06:45:34 +01:00

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

Crash by cyclic use of Boolean Modifier #32866