summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPablo Neira Ayuso <pablo@netfilter.org>2013-09-14 20:48:39 +0200
committerPablo Neira Ayuso <pablo@netfilter.org>2013-12-30 23:50:46 +0100
commit4c4bcbcd2523da740ed02021e51cb20b14fae153 (patch)
treeaaf2454e110b37bb6c334ea8d42c4db38fd49664
parent84909d171585d77fe769f03e2b1b96eab0aa0213 (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.c12
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;