[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