[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [23541] trunk/blender/source/blender/ windowmanager/intern/wm_event_system.c: Fix potential crash for file operators that don't define

[Brecht Van Lommel]

Revision: 23541
          
http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=23541
Author:   blendix
Date:     2009-09-28 18:11:06 +0200 (Mon, 28 Sep 2009)

Log Message:
-----------
Fix potential crash for file operators that don't define
a "path" property, now it will just give a warning.

Modified Paths:
--------------
    trunk/blender/source/blender/windowmanager/intern/wm_event_system.c

Modified: trunk/blender/source/blender/windowmanager/intern/wm_event_system.c
===================================================================
--- trunk/blender/source/blender/windowmanager/intern/wm_event_system.c 
2009-09-28 16:05:34 UTC (rev 23540)
+++ trunk/blender/source/blender/windowmanager/intern/wm_event_system.c 
2009-09-28 16:11:06 UTC (rev 23541)
@@ -943,7 +943,7 @@
                                        /* XXX also extension code in 
image-save doesnt work for this yet */
                                        if(strncmp(handler->op->type->name, 
"Save", 4)==0) {
                                                /* this gives ownership to 
pupmenu */
-                                               uiPupMenuSaveOver(C, 
handler->op, path);
+                                               uiPupMenuSaveOver(C, 
handler->op, (path)? path: "");
                                        }
                                        else {
                                                int retval= 
handler->op->type->exec(C, handler->op);
@@ -961,7 +961,8 @@
                                        WM_operator_free(handler->op);
                                
                                wm_event_free_handler(handler);
-                               MEM_freeN(path);
+                               if(path)
+                                       MEM_freeN(path);
                                
                                action= WM_HANDLER_BREAK;
                        }


_______________________________________________
Bf-blender-cvs mailing list
Bf-blender-cvs@blender.org
http://lists.blender.org/mailman/listinfo/bf-blender-cvs