diff options
author | Daniel Borkmann <dborkman@redhat.com> | 2014-12-09 10:43:31 +0100 |
---|---|---|
committer | Jesper Dangaard Brouer <brouer@redhat.com> | 2014-12-09 10:43:31 +0100 |
commit | 909f2dfae2bf112332125ba2512b94df9c008da9 (patch) | |
tree | d71eab5f0b8d3352fbafa43b20c8e664428827d6 | |
parent | b18edce035ad01cd1828573275cbf221a3ac02c6 (diff) | |
download | ipvsadm-909f2dfae2bf112332125ba2512b94df9c008da9.tar.gz |
ipvsadm: fix compile warning in print_largenum
ipvsadm.c: In function ‘print_largenum’:
ipvsadm.c:1445:3: warning: field width specifier ‘*’ expects argument of
type ‘int’, but argument 2 has type ‘size_t’ [-Wformat=]
printf("%*llu", len <= 8 ? 9 : len + 1, i);
^
Fix by reducing the scope and use int for len instead.
Signed-off-by: Daniel Borkmann <dborkman@redhat.com>
Acked-by: Simon Horman <horms@verge.net.au>
Acked-by: Julian Anastasov <ja@ssi.bg>
Signed-off-by: Jesper Dangaard Brouer <brouer@redhat.com>
-rw-r--r-- | ipvsadm.c | 8 |
1 files changed, 4 insertions, 4 deletions
@@ -1437,15 +1437,15 @@ static inline char *fwd_switch(unsigned flags) static void print_largenum(unsigned long long i, unsigned int format) { - char mytmp[32]; - size_t len; - if (format & FMT_EXACT) { + char mytmp[32]; + int len; + len = snprintf(mytmp, 32, "%llu", i); printf("%*llu", len <= 8 ? 9 : len + 1, i); return; } - + if (i < 100000000) /* less than 100 million */ printf("%9llu", i); else if (i < 1000000000) /* less than 1 billion */ |