netfilter: Fix format string of nfnetlink_queue proc file
authorRichard Weinberger <richard@nod.at>
Sun, 12 Apr 2015 22:52:38 +0000 (00:52 +0200)
committerDavid S. Miller <davem@davemloft.net>
Mon, 13 Apr 2015 20:35:16 +0000 (16:35 -0400)
The printed values are all of type unsigned integer, therefore use
%u instead of %d. Otherwise an user can face negative values.

Fixes:
$ cat /proc/net/netfilter/nfnetlink_queue
    0  29508   278 2 65531     0 2004213241 -2129885586  1
    1 -27747     0 2 65531     0     0        0  1
    2 -27748     0 2 65531     0     0        0  1

Signed-off-by: Richard Weinberger <richard@nod.at>
Acked-by: Pablo Neira Ayuso <pablo@netfilter.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/netfilter/nfnetlink_queue_core.c

index 49267f8e33b15199123541289be629416cea1a17..0b98c74202390ae79598ceb955360f937bb9556d 100644 (file)
@@ -1251,7 +1251,7 @@ static int seq_show(struct seq_file *s, void *v)
 {
        const struct nfqnl_instance *inst = v;
 
-       seq_printf(s, "%5d %6d %5d %1d %5d %5d %5d %8d %2d\n",
+       seq_printf(s, "%5u %6u %5u %1u %5u %5u %5u %8u %2d\n",
                   inst->queue_num,
                   inst->peer_portid, inst->queue_total,
                   inst->copy_mode, inst->copy_range,