Fix #31288: Blender crash when select a material
authorSergey Sharybin <sergey.vfx@gmail.com>
Fri, 4 May 2012 11:49:58 +0000 (11:49 +0000)
committerSergey Sharybin <sergey.vfx@gmail.com>
Fri, 4 May 2012 11:49:58 +0000 (11:49 +0000)
Quite the same issue as render crash on missed shader groups, needed a NULL-check
in node exec function as well.

source/blender/nodes/shader/nodes/node_shader_common.c
source/blender/nodes/texture/nodes/node_texture_common.c

index bd31f20b18c168ec71e57505944c58c788dd1eee..df369482a2ed64d4b16c8642b41cc8bb8dfbea39 100644 (file)
@@ -124,6 +124,9 @@ static void group_execute(void *data, int thread, struct bNode *node, void *node
        bNodeTreeExec *exec= (bNodeTreeExec*)nodedata;
        bNodeThreadStack *nts;
        
+       if (!exec)
+               return;
+       
        /* XXX same behavior as trunk: all nodes inside group are executed.
         * it's stupid, but just makes it work. compo redesign will do this better.
         */
index 2a9107f949820911fd2a5d025106c57c7684d6e6..1eaf9b2b0fed2db12305c4a30af9b978de003e1f 100644 (file)
@@ -110,6 +110,9 @@ static void group_execute(void *data, int thread, struct bNode *node, void *node
        bNodeTreeExec *exec= (bNodeTreeExec*)nodedata;
        bNodeThreadStack *nts;
        
+       if (!exec)
+               return;
+       
        /* XXX same behavior as trunk: all nodes inside group are executed.
         * it's stupid, but just makes it work. compo redesign will do this better.
         */