On 11.09.2017 19:20, Eric Blake wrote: > hmp() passes its string argument through the sprintf() family; > with a proper attribute, gcc -Wformat warns us when we do something > dangerous like passing a non-constant format string. Fortunately, > all our strings were safe, but checking whether the string can > contain an unintended % is easy to avoid and therefore worth doing. > > Signed-off-by: Eric Blake <ebl...@redhat.com> > > --- > v7: add GCC_FMT_ATTR here, drop R-b > --- > tests/libqtest.h | 8 ++++---- > tests/test-hmp.c | 4 ++-- > 2 files changed, 6 insertions(+), 6 deletions(-)
Reviewed-by: Thomas Huth <th...@redhat.com>