Fix format string warnings (gcc) by using string literals.
authorNicholas Bishop <nicholasbishop@gmail.com>
Mon, 19 Mar 2012 21:38:35 +0000 (21:38 +0000)
committerNicholas Bishop <nicholasbishop@gmail.com>
Mon, 19 Mar 2012 21:38:35 +0000 (21:38 +0000)
gcc 4.6 was giving warnings like this:
"warning: format not a string literal and no format arguments [-Wformat-security]"

source/blender/blenkernel/intern/DerivedMesh.c
source/blender/blenkernel/intern/cloth.c
source/blender/editors/armature/meshlaplacian.c
source/blender/modifiers/intern/MOD_boolean.c
source/blender/modifiers/intern/MOD_decimate.c
source/blender/modifiers/intern/MOD_meshdeform.c

index e533fa77135996bd2b5ee75c1566204c83e5431b..bf3f461d3f533f95de45879932dd30d218d852e8 100644 (file)
@@ -1474,7 +1474,7 @@ static void mesh_calc_modifiers(Scene *scene, Object *ob, float (*inputVertexCos
                if(!modifier_isEnabled(scene, md, required_mode)) continue;
                if(mti->type == eModifierTypeType_OnlyDeform && !useDeform) continue;
                if((mti->flags & eModifierTypeFlag_RequiresOriginalData) && dm) {
-                       modifier_setError(md, TIP_("Modifier requires original data, bad stack position."));
+                       modifier_setError(md, "%s", TIP_("Modifier requires original data, bad stack position."));
                        continue;
                }
                if(sculpt_mode && (!has_multires || multires_applied)) {
@@ -1487,7 +1487,7 @@ static void mesh_calc_modifiers(Scene *scene, Object *ob, float (*inputVertexCos
                        unsupported|= multires_applied;
 
                        if(unsupported) {
-                               modifier_setError(md, TIP_("Not supported in sculpt mode."));
+                               modifier_setError(md, "%s", TIP_("Not supported in sculpt mode."));
                                continue;
                        }
                }
@@ -1841,7 +1841,7 @@ int editbmesh_modifier_is_enabled(Scene *scene, ModifierData *md, DerivedMesh *d
 
        if(!modifier_isEnabled(scene, md, required_mode)) return 0;
        if((mti->flags & eModifierTypeFlag_RequiresOriginalData) && dm) {
-               modifier_setError(md, TIP_("Modifier requires original data, bad stack position."));
+               modifier_setError(md, "%s", TIP_("Modifier requires original data, bad stack position."));
                return 0;
        }
        
index d41b3684d9f4d0f9e2c253da602d94a53bcb3707..3fb8a3501d05f249a9da8f62af1700f664b5d167 100644 (file)
@@ -829,7 +829,7 @@ static int cloth_from_object(Object *ob, ClothModifierData *clmd, DerivedMesh *d
                clmd->clothObject->edgehash = NULL;
        }
        else if (!clmd->clothObject) {
-               modifier_setError(&(clmd->modifier), TIP_("Out of memory on allocating clmd->clothObject."));
+               modifier_setError(&(clmd->modifier), "%s", TIP_("Out of memory on allocating clmd->clothObject."));
                return 0;
        }
 
@@ -894,7 +894,7 @@ static int cloth_from_object(Object *ob, ClothModifierData *clmd, DerivedMesh *d
        if ( !cloth_build_springs ( clmd, dm ) )
        {
                cloth_free_modifier ( clmd );
-               modifier_setError(&(clmd->modifier), TIP_("Can't build springs."));
+               modifier_setError(&(clmd->modifier), "%s", TIP_("Can't build springs."));
                printf("cloth_free_modifier cloth_build_springs\n");
                return 0;
        }
@@ -940,7 +940,7 @@ static void cloth_from_mesh ( ClothModifierData *clmd, DerivedMesh *dm )
        if ( clmd->clothObject->verts == NULL )
        {
                cloth_free_modifier ( clmd );
-               modifier_setError(&(clmd->modifier), TIP_("Out of memory on allocating clmd->clothObject->verts."));
+               modifier_setError(&(clmd->modifier), "%s", TIP_("Out of memory on allocating clmd->clothObject->verts."));
                printf("cloth_free_modifier clmd->clothObject->verts\n");
                return;
        }
@@ -951,7 +951,7 @@ static void cloth_from_mesh ( ClothModifierData *clmd, DerivedMesh *dm )
        if ( clmd->clothObject->mfaces == NULL )
        {
                cloth_free_modifier ( clmd );
-               modifier_setError(&(clmd->modifier), TIP_("Out of memory on allocating clmd->clothObject->mfaces."));
+               modifier_setError(&(clmd->modifier), "%s", TIP_("Out of memory on allocating clmd->clothObject->mfaces."));
                printf("cloth_free_modifier clmd->clothObject->mfaces\n");
                return;
        }
index 17c6e25b52b550bfb2415ae4583c384b9c319b8d..de9b4a890cba0637ce73dfa9e9bd8b049484a41b 100644 (file)
@@ -1716,7 +1716,7 @@ static void meshdeform_matrix_solve(MeshDeformModifierData *mmd, MeshDeformBind
                        }
                }
                else {
-                       modifier_setError(&mmd->modifier, TIP_("Failed to find bind solution (increase precision?)."));
+                       modifier_setError(&mmd->modifier, "%s", TIP_("Failed to find bind solution (increase precision?)."));
                        error("Mesh Deform: failed to find bind solution.");
                        break;
                }
index 3bf30e5fa144fa147e4ec11bd93384880054d814..2b3c1f00541799be9afb51cd1d5bc912baa5a7d2 100644 (file)
@@ -156,7 +156,7 @@ static DerivedMesh *applyModifier(ModifierData *md, Object *ob,
                if(result)
                        return result;
                else
-                       modifier_setError(md, TIP_("Can't execute boolean operation."));
+                       modifier_setError(md, "%s", TIP_("Can't execute boolean operation."));
        }
        
        return derivedData;
index 5fc3c506d36fc19ec6095b6be134da6554b363f0..3c3381dc36b5cb5d1dfdffe0985506663a0c72d4 100644 (file)
@@ -98,7 +98,7 @@ static DerivedMesh *applyModifier(ModifierData *md, Object *UNUSED(ob),
        }
 
        if(numTris<3) {
-               modifier_setError(md, TIP_("Modifier requires more than 3 input faces (triangles)."));
+               modifier_setError(md, "%s", TIP_("Modifier requires more than 3 input faces (triangles)."));
                dm = CDDM_copy(dm);
                return dm;
        }
@@ -173,12 +173,12 @@ static DerivedMesh *applyModifier(ModifierData *md, Object *UNUSED(ob),
                        CDDM_calc_edges_tessface(result);
                }
                else
-                       modifier_setError(md, TIP_("Out of memory."));
+                       modifier_setError(md, "%s", TIP_("Out of memory."));
 
                LOD_FreeDecimationData(&lod);
        }
        else
-               modifier_setError(md, TIP_("Non-manifold mesh as input."));
+               modifier_setError(md, "%s", TIP_("Non-manifold mesh as input."));
 
        MEM_freeN(lod.vertex_buffer);
        MEM_freeN(lod.vertex_normal_buffer);
index afee0e2c79e2ea180af153935f1bb8533eac3ae4..c8ce1b9b88a413b10cb9ead2c0af0fe31141bc4c 100644 (file)
@@ -218,7 +218,7 @@ static void meshdeformModifier_do(
        }
        
        if(!cagedm) {
-               modifier_setError(md, TIP_("Can't get mesh from cage object."));
+               modifier_setError(md, "%s", TIP_("Can't get mesh from cage object."));
                return;
        }
 
@@ -255,7 +255,7 @@ static void meshdeformModifier_do(
                cagedm->release(cagedm);
                return;
        } else if (mmd->bindcagecos == NULL) {
-               modifier_setError(md, TIP_("Bind data missing."));
+               modifier_setError(md, "%s", TIP_("Bind data missing."));
                cagedm->release(cagedm);
                return;
        }