diff options
| author | Phil Sutter <phil@nwl.cc> | 2023-11-08 04:08:44 +0100 |
|---|---|---|
| committer | Phil Sutter <phil@nwl.cc> | 2023-11-09 15:55:30 +0100 |
| commit | 2eb823b520e1503cd3fbc2c66b8bc50beadac6b0 (patch) | |
| tree | 9ba3ba910c1d07fac5aa645dd02ab9077058b188 /iptables | |
| parent | 59062564de9d4a3a8ed8b2e0010338dc6d2f1408 (diff) | |
arptables: Fix --proto-type mask formatting
Arptables accepts numeric --proto-type values and masks in any numeral
system identified by (absence of) prefix. Yet it prints the mask value
in hex without '0x'-prefix, breaking save and restore the same way
numeric --h-type output did.
In theory, this could be fixed either by adding the missing prefix or
printing the mask in decimal (like most other builtin matches do), but
since the value is printed in hex with prefix already, align mask output
with that.
Also a day 1 bug and consistent with legacy, so no Fixes: tag here as
well.
Signed-off-by: Phil Sutter <phil@nwl.cc>
Acked-by: Florian Westphal <fw@strlen.de>
Diffstat (limited to 'iptables')
| -rw-r--r-- | iptables/nft-arp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/iptables/nft-arp.c b/iptables/nft-arp.c index 83aec500..38b2ab39 100644 --- a/iptables/nft-arp.c +++ b/iptables/nft-arp.c @@ -339,7 +339,7 @@ after_devdst: else printf("--proto-type 0x%x", tmp); if (fw->arp.arpro_mask != 65535) - printf("/%x", ntohs(fw->arp.arpro_mask)); + printf("/0x%x", ntohs(fw->arp.arpro_mask)); sep = " "; } } |