From 73a0c2ef36d15e8277f0e00337736668b325381a Mon Sep 17 00:00:00 2001 From: Phil Sutter Date: Fri, 15 Feb 2019 15:27:43 +0100 Subject: arptables: Print space before comma and counters Legacy arptables separates counters from rest of rule by ' , '. Assuming that scripts scraping 'arptables -vL' output match on this, make arptables-nft output conformant. Signed-off-by: Phil Sutter Signed-off-by: Pablo Neira Ayuso --- iptables/nft-arp.c | 2 +- .../testcases/arptables/0003-arptables-verbose-output_0 | 12 ++++++------ 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/iptables/nft-arp.c b/iptables/nft-arp.c index 637da427..a3715579 100644 --- a/iptables/nft-arp.c +++ b/iptables/nft-arp.c @@ -617,7 +617,7 @@ nft_arp_print_rule(struct nftnl_rule *r, unsigned int num, unsigned int format) print_matches_and_target(&cs, format); if (!(format & FMT_NOCOUNTS)) { - printf(", pcnt="); + printf(" , pcnt="); xtables_print_num(cs.counters.pcnt, format | FMT_NOTABLE); printf("-- bcnt="); xtables_print_num(cs.counters.bcnt, format | FMT_NOTABLE); diff --git a/iptables/tests/shell/testcases/arptables/0003-arptables-verbose-output_0 b/iptables/tests/shell/testcases/arptables/0003-arptables-verbose-output_0 index 35126fa7..3a9807a1 100755 --- a/iptables/tests/shell/testcases/arptables/0003-arptables-verbose-output_0 +++ b/iptables/tests/shell/testcases/arptables/0003-arptables-verbose-output_0 @@ -36,16 +36,16 @@ diff -u -Z <(echo -e "$VOUT5") <($XT_MULTI arptables -v -A OUTPUT $RULE5) diff -u -Z <(echo -e "$VOUT6") <($XT_MULTI arptables -v -A foo $RULE6) EXPECT='Chain INPUT (policy ACCEPT 0 packets, 0 bytes) --j ACCEPT -i eth23 -o *, pcnt=0 -- bcnt=0 --i eth23 -o *, pcnt=0 -- bcnt=0 --j MARK -i eth23 -o * --set-mark 42, pcnt=0 -- bcnt=0 +-j ACCEPT -i eth23 -o * , pcnt=0 -- bcnt=0 +-i eth23 -o * , pcnt=0 -- bcnt=0 +-j MARK -i eth23 -o * --set-mark 42 , pcnt=0 -- bcnt=0 Chain OUTPUT (policy ACCEPT 0 packets, 0 bytes) --j CLASSIFY -i * -o eth23 --set-class 23:42, pcnt=0 -- bcnt=0 --j foo -i * -o eth23, pcnt=0 -- bcnt=0 +-j CLASSIFY -i * -o eth23 --set-class 23:42 , pcnt=0 -- bcnt=0 +-j foo -i * -o eth23 , pcnt=0 -- bcnt=0 Chain foo (1 references) --j mangle -i * -o eth23 --mangle-ip-s 10.0.0.1, pcnt=0 -- bcnt=0' +-j mangle -i * -o eth23 --mangle-ip-s 10.0.0.1 , pcnt=0 -- bcnt=0' diff -u -Z <(echo -e "$EXPECT") <($XT_MULTI arptables -v -n -L) -- cgit v1.2.3