diff options
author | Theodore Ts'o <tytso@mit.edu> | 2009-01-20 13:05:25 -0500 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2009-01-20 13:05:25 -0500 |
commit | 5299580c1ced39e7a6d7ac2717a3d6a3cab299b0 (patch) | |
tree | 386dba123dc885ca156203e93f9520c5e4b22161 /lib | |
parent | 31e2a75484816db0e5e68afacb13beef7e751061 (diff) | |
download | e2fsprogs-5299580c1ced39e7a6d7ac2717a3d6a3cab299b0.tar.gz |
Use format strings directly to prevent -Werror=format-security failures
Gcc is too stupid to realize that:
const char *usage="String which has no percent signs";
com_err(progname, 0, usage);
is OK. I refuse to bow to stupidity with:
com_err(progname, 0, "%s", usage);
but I will use the string directly for the sake of people who like to
build with -Werror=format-security.
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'lib')
-rw-r--r-- | lib/ss/test_ss.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/ss/test_ss.c b/lib/ss/test_ss.c index aba7cbcd..41d84ea7 100644 --- a/lib/ss/test_ss.c +++ b/lib/ss/test_ss.c @@ -87,7 +87,6 @@ int main(int argc, char **argv) char *cmd_file = 0; int sci_idx; int exit_status = 0; - const char *usage = "Usage: test_ss [-R request] [-f cmd_file]"; while ((c = getopt (argc, argv, "wR:f:")) != EOF) { switch (c) { @@ -98,7 +97,8 @@ int main(int argc, char **argv) cmd_file = optarg; break; default: - com_err(argv[0], 0, usage); + com_err(argv[0], 0, "Usage: test_ss [-R request] " + "[-f cmd_file]"); exit(1); } } |