[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [17974] trunk/blender/source/blender/ readblenfile/intern/BLO_readblenfile.c: Bugfix since rev 2, BLO_setversionnumber() was using sizeof(array), which gives the sizeof the pointer, not the array.

Campbell Barton ideasman42 at gmail.com
Sat Dec 20 18:09:45 CET 2008


Revision: 17974
          http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=17974
Author:   campbellbarton
Date:     2008-12-20 18:09:45 +0100 (Sat, 20 Dec 2008)

Log Message:
-----------
Bugfix since rev 2, BLO_setversionnumber() was using sizeof(array), which gives the sizeof the pointer, not the array.
On a 64bit system I assume this would set 8 chars of a 4 char array.

Turns out this isnt such a problem since BLO_setversionnumber isn't used anymore but way as well commit.

Revision Links:
--------------
    http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=2

Modified Paths:
--------------
    trunk/blender/source/blender/readblenfile/intern/BLO_readblenfile.c

Modified: trunk/blender/source/blender/readblenfile/intern/BLO_readblenfile.c
===================================================================
--- trunk/blender/source/blender/readblenfile/intern/BLO_readblenfile.c	2008-12-20 15:42:48 UTC (rev 17973)
+++ trunk/blender/source/blender/readblenfile/intern/BLO_readblenfile.c	2008-12-20 17:09:45 UTC (rev 17974)
@@ -67,7 +67,7 @@
  */
 void BLO_setversionnumber(char array[4], int version)
 {
-	memset(array, 0, sizeof(array));
+	memset(array, 0, sizeof(char)*4);
 
 	array[1] = version / 100;
 	array[2] = version % 100;





More information about the Bf-blender-cvs mailing list