[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 brecht at blender.org
Mon Sep 28 18:11:06 CEST 2009


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;
 			}





More information about the Bf-blender-cvs mailing list