glsl: s/sprintf/_mesa_snprintf/
authorVinson Lee <vlee@vmware.com>
Sun, 2 May 2010 05:01:28 +0000 (22:01 -0700)
committerVinson Lee <vlee@vmware.com>
Sun, 2 May 2010 05:01:28 +0000 (22:01 -0700)
src/mesa/shader/slang/slang_codegen.c
src/mesa/shader/slang/slang_emit.c
src/mesa/shader/slang/slang_ir.c
src/mesa/shader/slang/slang_label.c
src/mesa/shader/slang/slang_link.c
src/mesa/shader/slang/slang_print.c
src/mesa/shader/slang/slang_utility.c

index fa79632c18bbf80b055e60eacaa80ea44140bf30..27ac1da9ad8aafb49c5b9f40fc573a4b5996d014 100644 (file)
@@ -1560,7 +1560,7 @@ _slang_gen_function_call(slang_assemble_ctx *A, slang_function *fun,
    char name[200];
 
    prevFuncEndLabel = A->curFuncEndLabel;
-   sprintf(name, "__endOfFunc_%s_", (char *) fun->header.a_name);
+   _mesa_snprintf(name, sizeof(name), "__endOfFunc_%s_", (char *) fun->header.a_name);
    A->curFuncEndLabel = _slang_label_new(name);
    assert(A->curFuncEndLabel);
 
index 7c0ea0c11484290c19661adfb0a07206e3448838..974c4a31315470f8c36a03999d5afa1478d8f0d1 100644 (file)
@@ -652,30 +652,30 @@ storage_annotation(const slang_ir_node *n, const struct gl_program *prog)
       if (st->Index >= 0) {
          const GLfloat *val = prog->Parameters->ParameterValues[st->Index];
          if (st->Swizzle == SWIZZLE_NOOP)
-            sprintf(s, "{%g, %g, %g, %g}", val[0], val[1], val[2], val[3]);
+            _mesa_snprintf(s, sizeof(s), "{%g, %g, %g, %g}", val[0], val[1], val[2], val[3]);
          else {
-            sprintf(s, "%g", val[GET_SWZ(st->Swizzle, 0)]);
+            _mesa_snprintf(s, sizeof(s), "%g", val[GET_SWZ(st->Swizzle, 0)]);
          }
       }
       break;
    case PROGRAM_TEMPORARY:
       if (n->Var)
-         sprintf(s, "%s", (char *) n->Var->a_name);
+         _mesa_snprintf(s, sizeof(s), "%s", (char *) n->Var->a_name);
       else
-         sprintf(s, "t[%d]", st->Index);
+         _mesa_snprintf(s, sizeof(s), "t[%d]", st->Index);
       break;
    case PROGRAM_STATE_VAR:
    case PROGRAM_UNIFORM:
-      sprintf(s, "%s", prog->Parameters->Parameters[st->Index].Name);
+      _mesa_snprintf(s, sizeof(s), "%s", prog->Parameters->Parameters[st->Index].Name);
       break;
    case PROGRAM_VARYING:
-      sprintf(s, "%s", prog->Varying->Parameters[st->Index].Name);
+      _mesa_snprintf(s, sizeof(s), "%s", prog->Varying->Parameters[st->Index].Name);
       break;
    case PROGRAM_INPUT:
-      sprintf(s, "input[%d]", st->Index);
+      _mesa_snprintf(s, sizeof(s), "input[%d]", st->Index);
       break;
    case PROGRAM_OUTPUT:
-      sprintf(s, "output[%d]", st->Index);
+      _mesa_snprintf(s, sizeof(s), "output[%d]", st->Index);
       break;
    default:
       s[0] = 0;
@@ -752,9 +752,8 @@ instruction_annotation(gl_inst_opcode opcode, char *dstAnnot,
    }
 
    s = (char *) malloc(len);
-   sprintf(s, "%s = %s %s %s %s", dstAnnot,
-           srcAnnot0, operator, srcAnnot1, srcAnnot2);
-   assert(strlen(s) < len);
+   _mesa_snprintf(s, len, "%s = %s %s %s %s", dstAnnot,
+                  srcAnnot0, operator, srcAnnot1, srcAnnot2);
 
    free(dstAnnot);
    free(srcAnnot0);
@@ -2274,11 +2273,11 @@ emit_var_decl(slang_emit_info *emitInfo, slang_ir_node *n)
    if (emitInfo->EmitComments) {
       /* emit NOP with comment describing the variable's storage location */
       char s[1000];
-      sprintf(s, "TEMP[%d]%s = variable %s (size %d)",
-              n->Store->Index,
-              _mesa_swizzle_string(n->Store->Swizzle, 0, GL_FALSE), 
-              (n->Var ? (char *) n->Var->a_name : "anonymous"),
-              n->Store->Size);
+      _mesa_snprintf(s, sizeof(s), "TEMP[%d]%s = variable %s (size %d)",
+                     n->Store->Index,
+                     _mesa_swizzle_string(n->Store->Swizzle, 0, GL_FALSE), 
+                     (n->Var ? (char *) n->Var->a_name : "anonymous"),
+                     n->Store->Size);
       emit_comment(emitInfo, s);
    }
    return NULL;
index 62603503dd9ad32d4a278dc4d3d83210e970d7ed..c223004b22b9bbcfb4a126e5d80b6c116ca4ee99 100644 (file)
@@ -340,13 +340,13 @@ storage_string(const slang_ir_storage *st)
    assert(Elements(files) == PROGRAM_FILE_MAX);
 #if 0
    if (st->Size == 1)
-      sprintf(s, "%s[%d]", files[st->File], st->Index);
+      _mesa_snprintf(s, "%s[%d]", files[st->File], st->Index);
    else
-      sprintf(s, "%s[%d..%d]", files[st->File], st->Index,
-              st->Index + st->Size - 1);
+      _mesa_snprintf(s, "%s[%d..%d]", files[st->File], st->Index,
+                     st->Index + st->Size - 1);
 #endif
    assert(st->File < (GLint) (sizeof(files) / sizeof(files[0])));
-   sprintf(s, "%s[%d]", files[st->File], st->Index);
+   _mesa_snprintf(s, sizeof(s), "%s[%d]", files[st->File], st->Index);
    return s;
 }
 
index 225612a936940d980da13472ca309d22b39b3fe9..8e3a8ebc1aaae3c6987c01eac2e77cff65c81afb 100644 (file)
@@ -37,7 +37,7 @@ _slang_label_new_unique(const char *name)
          free(l);
          return NULL;
       }
-      sprintf(l->Name, "%s_%d", name, id);
+      _mesa_snprintf(l->Name, strlen(name) + 10, "%s_%d", name, id);
       id++;
       l->Location = -1;
    }
index 47bc93397163f06cf49cb6b8d2e1fc68bfbe13b3..b16778f8ad4ce42449296e02aadf14b25a740121 100644 (file)
@@ -370,8 +370,9 @@ link_uniform_vars(GLcontext *ctx,
          GLuint newSampNum = *numSamplers;
          if (newSampNum >= ctx->Const.MaxTextureImageUnits) {
             char s[100];
-            sprintf(s, "Too many texture samplers (%u, max is %u)",
-                   newSampNum, ctx->Const.MaxTextureImageUnits);
+            _mesa_snprintf(s, sizeof(s),
+                           "Too many texture samplers (%u, max is %u)",
+                           newSampNum, ctx->Const.MaxTextureImageUnits);
             link_error(shProg, s);
             return GL_FALSE;
          }
index 3c75523c42fbdfd3e04eb5942f721bc9df237022..6b34f395fdfa00fe2084f6f96c145200427e4d2e 100644 (file)
@@ -813,7 +813,7 @@ static const char *
 slang_fq_type_string(const slang_fully_specified_type *t)
 {
    static char str[1000];
-   sprintf(str, "%s %s", slang_type_qual_string(t->qualifier),
+   _mesa_snprintf(str, sizeof(str), "%s %s", slang_type_qual_string(t->qualifier),
       slang_type_string(t->specifier.type));
    return str;
 }
@@ -832,9 +832,9 @@ static char *
 slang_var_string(const slang_variable *v)
 {
    static char str[1000];
-   sprintf(str, "%s : %s",
-           (char *) v->a_name,
-           slang_fq_type_string(&v->type));
+   _mesa_snprintf(str, sizeof(str), "%s : %s",
+                  (char *) v->a_name,
+                  slang_fq_type_string(&v->type));
    return str;
 }
 #endif
index e77404f692861ac92a47f149b0224aa4af5aabfc..c1d57409a436e334812a7276e294b6c35d701be5 100644 (file)
@@ -120,7 +120,7 @@ slang_string_pushi (slang_string *self, GLint i)
 {
    char buffer[12];
 
-   sprintf (buffer, "%d", i);
+   _mesa_snprintf (buffer, sizeof(buffer), "%d", i);
    slang_string_pushs (self, buffer, strlen (buffer));
 }