Message ID | 1500645206-13559-2-git-send-email-armbru@redhat.com |
---|---|
State | New |
Headers | show |
On Fri, Jul 21, 2017 at 03:53:18PM +0200, Markus Armbruster wrote: > From: Eric Blake <eblake@redhat.com> > > The next patch will add __attribute__((__format__)) to hmp(), which > in turn causes gcc to warn about non-literal format strings. Rather > than risk an arbitrary string containing % being mis-handled, always > pass variable strings along with a %s format. It also makes it > easier to prove correctness locally, rather than auditing all the > source strings. > > Signed-off-by: Eric Blake <eblake@redhat.com> > Reviewed-by: Markus Armbruster <armbru@redhat.com> > Message-Id: <20170720214008.28494-4-eblake@redhat.com> > Signed-off-by: Markus Armbruster <armbru@redhat.com> > --- > tests/test-hmp.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
diff --git a/tests/test-hmp.c b/tests/test-hmp.c index d77b3c8..0af0664 100644 --- a/tests/test-hmp.c +++ b/tests/test-hmp.c @@ -80,7 +80,7 @@ static void test_commands(void) if (verbose) { fprintf(stderr, "\t%s\n", hmp_cmds[i]); } - response = hmp(hmp_cmds[i]); + response = hmp("%s", hmp_cmds[i]); g_free(response); } @@ -103,7 +103,7 @@ static void test_info_commands(void) if (verbose) { fprintf(stderr, "\t%s\n", info); } - resp = hmp(info); + resp = hmp("%s", info); g_free(resp); /* And move forward to the next line */ info = strchr(endp + 1, '\n');