diff options
| author | Niels de Vos <ndevos@redhat.com> | 2014-08-28 10:46:50 +0200 |
|---|---|---|
| committer | Vijay Bellur <vbellur@redhat.com> | 2014-08-29 10:01:50 -0700 |
| commit | 370592cd702a2c135ba4accfcbe9e1a63c3b6166 (patch) | |
| tree | f8d87217ae85a56a03414d66fd646c9dbfba6e5d /cli/src/cli-rpc-ops.c | |
| parent | ea96a0abcd23bd729d8167ec2b88af12d4dbb62c (diff) | |
build: fixes for compiling with -Werror=format-security
Nightly builds on Fedora Rawhide (x86_64 and i686) started to fail with the
following error:
cli-rpc-ops.c: In function 'gf_cli_get_vol_opt_cbk':
cli-rpc-ops.c:9339:25: error: format not a string literal and no format arguments [-Werror=format-security]
cli_err (msg);
^
The cli_err() should be called with a format-string like "%s" to prevent
unexpected errors.
Change-Id: I6a287795fb8b7a305c8f14323456eeabf68d100c
BUG: 1134773
Signed-off-by: Niels de Vos <ndevos@redhat.com>
Reviewed-on: http://review.gluster.org/8560
Reviewed-by: Harshavardhana <harsha@harshavardhana.net>
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Humble Devassy Chirammal <humble.devassy@gmail.com>
Reviewed-by: Kaleb KEITHLEY <kkeithle@redhat.com>
Diffstat (limited to 'cli/src/cli-rpc-ops.c')
| -rw-r--r-- | cli/src/cli-rpc-ops.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/cli/src/cli-rpc-ops.c b/cli/src/cli-rpc-ops.c index ed3faf70c39..ba8ba6810d6 100644 --- a/cli/src/cli-rpc-ops.c +++ b/cli/src/cli-rpc-ops.c @@ -9336,7 +9336,7 @@ gf_cli_get_vol_opt_cbk (struct rpc_req *req, struct iovec *iov, int count, "Error outputting to xml"); } } else { - cli_err (msg); + cli_err ("%s", msg); } ret = 0; /* setting ret to 0 ensures we do not end up displaying double error messages */ |