[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [10879] trunk/blender/source/blender/ blenkernel/intern/exotic.c: filelen is set to the return result of BLI_filesize

Kent Mein mein at cs.umn.edu
Tue Jun 5 17:43:20 CEST 2007


Revision: 10879
          http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=10879
Author:   sirdude
Date:     2007-06-05 17:43:20 +0200 (Tue, 05 Jun 2007)

Log Message:
-----------
filelen is set to the return result of BLI_filesize
which can return -1.  So lets check for -1 before calling malloc
and doing more work.

Another coverity fix.

Kent

Modified Paths:
--------------
    trunk/blender/source/blender/blenkernel/intern/exotic.c

Modified: trunk/blender/source/blender/blenkernel/intern/exotic.c
===================================================================
--- trunk/blender/source/blender/blenkernel/intern/exotic.c	2007-06-05 15:11:39 UTC (rev 10878)
+++ trunk/blender/source/blender/blenkernel/intern/exotic.c	2007-06-05 15:43:20 UTC (rev 10879)
@@ -1246,7 +1246,12 @@
 		error("Can't read file\n");
 		return;
 	}
+
 	filelen= BLI_filesize(file);
+	if(filelen < 1) {
+		close(file);
+		return;
+	}
 	
 	maindata= MEM_mallocN(filelen, "leesInventor");
 	read(file, maindata, filelen);





More information about the Bf-blender-cvs mailing list