Blender Git Loki
Git Commits -> Revision 807ad1f
Revision 807ad1f by Sergey Sharybin (master) July 13, 2012, 13:47 (GMT) |
Fix #32087: Crash while changing values in comp editor (bt and blender included) Issue was caused by threading conflict between compositor output node which is freeing buffers used by render result image and image draw code which could use buffers at the same time as compositor frees this buffers. Solved by adding adding lock around viewer image invalidation and image drawing. Use renamed LOCK_PREVIEW mutex for this, which si not called LOCK_DRAW_IMAGE. With new compositor locking for preview is not needed so it could be removed. Added the same lock around viewer operation which also frees buffers used by viewer image. It's actually quite difficult to check whether this is indeed needed. This code seems to be using acquire/release technique, but somehow acquiring ImBuf before invalidating it in compositor operation doesn't resolve the issue, so probably it's not actually locking acquire and things should be checked deeper. |
Commit Details:
Full Hash: 807ad1f0e0de49711d58ade2b660e74ec5452b2b
SVN Revision: 48893
Parent Commit: a5e6d73
Lines Changed: +31, -17
7 Modified Paths:
/source/blender/blenlib/BLI_threads.h (+1, -1) (Diff)
/source/blender/blenlib/intern/threads.c (+5, -5) (Diff)
/source/blender/compositor/operations/COM_CompositorOperation.cpp (+4, -0) (Diff)
/source/blender/compositor/operations/COM_ViewerBaseOperation.cpp (+4, -0) (Diff)
/source/blender/editors/space_image/image_draw.c (+15, -2) (Diff)
/source/blender/editors/space_node/node_draw.c (+0, -7) (Diff)
/source/blender/nodes/composite/node_composite_util.c (+2, -2) (Diff)
/source/blender/blenlib/intern/threads.c (+5, -5) (Diff)
/source/blender/compositor/operations/COM_CompositorOperation.cpp (+4, -0) (Diff)
/source/blender/compositor/operations/COM_ViewerBaseOperation.cpp (+4, -0) (Diff)
/source/blender/editors/space_image/image_draw.c (+15, -2) (Diff)
/source/blender/editors/space_node/node_draw.c (+0, -7) (Diff)
/source/blender/nodes/composite/node_composite_util.c (+2, -2) (Diff)