expprint: Fix format string warning
authorSimon Marchi <simon.marchi@ericsson.com>
Tue, 5 Sep 2017 07:00:42 +0000 (09:00 +0200)
committerSimon Marchi <simon.marchi@ericsson.com>
Tue, 5 Sep 2017 07:01:13 +0000 (09:01 +0200)
My compiler (gcc 5.4.0, clang 3.8) gives this warning:

/home/emaisin/src/binutils-gdb/gdb/expprint.c: In lambda function:
/home/emaisin/src/binutils-gdb/gdb/expprint.c:1055:35: error: format not a string literal and no format arguments [-Werror=format-security]
      fprintf_filtered (stream, mod);
                                   ^

Fix it by not using the passed string as the format string.

gdb/ChangeLog:

* expprint.c (dump_subexp_body_standard): Use constant format
string in fprintf_filtered call.

gdb/ChangeLog
gdb/expprint.c

index de1945a..19882f0 100644 (file)
@@ -1,3 +1,8 @@
+2017-09-05  Simon Marchi  <simon.marchi@ericsson.com>
+
+       * expprint.c (dump_subexp_body_standard): Use constant format
+       string in fprintf_filtered call.
+
 2017-09-04  John Baldwin  <jhb@FreeBSD.org>
 
        * configure.nat: Add "x86-nat.o x86-dregs.o" for NetBSD/amd64 and
index 2c16b49..9e04f24 100644 (file)
@@ -1052,7 +1052,7 @@ dump_subexp_body_standard (struct expression *exp,
            if (space)
              fputs_filtered (" ", stream);
            space = true;
-           fprintf_filtered (stream, mod);
+           fprintf_filtered (stream, "%s", mod);
          };
        if (flags & TYPE_INSTANCE_FLAG_CONST)
          print_one ("const");