glsl_compiler: Remove unused extra argument to printf in usage_fail
authorAlan Coopersmith <alan.coopersmith@oracle.com>
Sat, 23 May 2015 07:03:53 +0000 (00:03 -0700)
committerAlan Coopersmith <alan.coopersmith@oracle.com>
Fri, 5 Jun 2015 02:01:16 +0000 (19:01 -0700)
Flagged by Oracle's parfait static analyzer:

Error: Format string argument mismatch (CWE 628)
   In call to printf with format string "usage: %s [options] <file.vert | file.geom | file.frag>\n\nPossible options are:\n"
      Too many arguments for format string (got more than 1 arguments)
        at line 285 of src/glsl/main.cpp in function 'usage_fail'.

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
src/glsl/main.cpp

index ccac839964694c38b3b7cfcb3799a399b7a22f10..4b39c9e54c272770271175c8a33cad551f34f2d0 100644 (file)
@@ -282,7 +282,7 @@ usage_fail(const char *name)
       "usage: %s [options] <file.vert | file.geom | file.frag>\n"
       "\n"
       "Possible options are:\n";
-   printf(header, name, name);
+   printf(header, name);
    for (const struct option *o = compiler_opts; o->name != 0; ++o) {
       printf("    --%s\n", o->name);
    }