[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [20860] trunk/blender/source/blender/gpu/ intern/gpu_material.c: coverity issue CID: 228

Kent Mein mein at cs.umn.edu
Sat Jun 13 18:49:30 CEST 2009


Revision: 20860
          http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=20860
Author:   sirdude
Date:     2009-06-13 18:49:30 +0200 (Sat, 13 Jun 2009)

Log Message:
-----------
coverity issue CID: 228
Checker: DEADCODE (help)
File: base/src/source/blender/gpu/intern/gpu_material.c
Function: GPU_materials_free
Description: Conditional "ma != 0"

was looping over wrong variable I'm guessing a cut and
paste error from above.

Kent

Modified Paths:
--------------
    trunk/blender/source/blender/gpu/intern/gpu_material.c

Modified: trunk/blender/source/blender/gpu/intern/gpu_material.c
===================================================================
--- trunk/blender/source/blender/gpu/intern/gpu_material.c	2009-06-13 15:54:39 UTC (rev 20859)
+++ trunk/blender/source/blender/gpu/intern/gpu_material.c	2009-06-13 16:49:30 UTC (rev 20860)
@@ -1285,7 +1285,7 @@
 
 	GPU_material_free(&defmaterial);
 
-	for(ob=G.main->object.first; ma; ma=ma->id.next)
+	for(ob=G.main->object.first; ob; ob=ob->id.next)
 		GPU_lamp_free(ob);
 }
 





More information about the Bf-blender-cvs mailing list