diff options
author | Pablo Neira Ayuso <pablo@netfilter.org> | 2013-09-14 20:48:39 +0200 |
---|---|---|
committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2013-12-30 23:50:46 +0100 |
commit | 4c4bcbcd2523da740ed02021e51cb20b14fae153 (patch) | |
tree | aaf2454e110b37bb6c334ea8d42c4db38fd49664 | |
parent | 84909d171585d77fe769f03e2b1b96eab0aa0213 (diff) |
xtables: nft-arp: fix endianess in nft_arp_parse_payload
nft_arp_print_firewall already handles endianess appropriately.
This fixes the output of the option that allows you to match
ARP header fields.
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
-rw-r--r-- | iptables/nft-arp.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/iptables/nft-arp.c b/iptables/nft-arp.c index 09b939d0..6ec8a455 100644 --- a/iptables/nft-arp.c +++ b/iptables/nft-arp.c @@ -287,22 +287,22 @@ static void nft_arp_parse_payload(struct nft_rule_expr_iter *iter, switch (offset) { case offsetof(struct arphdr, ar_hrd): get_cmp_data(iter, &ar_hrd, sizeof(ar_hrd), &inv); - fw->arp.arhrd = htons(ar_hrd); - fw->arp.arhrd_mask = htons(0xffff); + fw->arp.arhrd = ar_hrd; + fw->arp.arhrd_mask = 0xffff; if (inv) fw->arp.invflags |= ARPT_INV_ARPHRD; break; case offsetof(struct arphdr, ar_pro): get_cmp_data(iter, &ar_pro, sizeof(ar_pro), &inv); - fw->arp.arpro = htons(ar_pro); - fw->arp.arpro_mask = htons(0xffff); + fw->arp.arpro = ar_pro; + fw->arp.arpro_mask = 0xffff; if (inv) fw->arp.invflags |= ARPT_INV_ARPPRO; break; case offsetof(struct arphdr, ar_op): get_cmp_data(iter, &ar_op, sizeof(ar_op), &inv); - fw->arp.arpop = htons(ar_op); - fw->arp.arpop_mask = htons(0xffff); + fw->arp.arpop = ar_op; + fw->arp.arpop_mask = 0xffff; if (inv) fw->arp.invflags |= ARPT_INV_ARPOP; break; |