[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [39047] trunk/blender/source/blender: fix for possible uninitialized RNA strings, when RNA_string_get property is not found, initialize the string to "".

Campbell Barton ideasman42 at gmail.com
Fri Aug 5 08:09:30 CEST 2011


Revision: 39047
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=39047
Author:   campbellbarton
Date:     2011-08-05 06:09:30 +0000 (Fri, 05 Aug 2011)
Log Message:
-----------
fix for possible uninitialized RNA strings, when RNA_string_get property is not found, initialize the string to "".

Modified Paths:
--------------
    trunk/blender/source/blender/makesrna/intern/rna_access.c
    trunk/blender/source/blender/windowmanager/intern/wm_operators.c

Modified: trunk/blender/source/blender/makesrna/intern/rna_access.c
===================================================================
--- trunk/blender/source/blender/makesrna/intern/rna_access.c	2011-08-05 06:06:15 UTC (rev 39046)
+++ trunk/blender/source/blender/makesrna/intern/rna_access.c	2011-08-05 06:09:30 UTC (rev 39047)
@@ -4007,10 +4007,13 @@
 {
 	PropertyRNA *prop= RNA_struct_find_property(ptr, name);
 
-	if(prop)
+	if(prop) {
 		RNA_property_string_get(ptr, prop, value);
-	else
+	}
+	else {
 		printf("RNA_string_get: %s.%s not found.\n", ptr->type->identifier, name);
+		value[0]= '\0';
+	}
 }
 
 char *RNA_string_get_alloc(PointerRNA *ptr, const char *name, char *fixedbuf, int fixedlen)

Modified: trunk/blender/source/blender/windowmanager/intern/wm_operators.c
===================================================================
--- trunk/blender/source/blender/windowmanager/intern/wm_operators.c	2011-08-05 06:06:15 UTC (rev 39046)
+++ trunk/blender/source/blender/windowmanager/intern/wm_operators.c	2011-08-05 06:09:30 UTC (rev 39047)
@@ -1621,7 +1621,6 @@
 	int idcode, totfiles=0;
 	short flag;
 
-	name[0] = '\0';
 	RNA_string_get(op->ptr, "filename", name);
 	RNA_string_get(op->ptr, "directory", dir);
 




More information about the Bf-blender-cvs mailing list