diff options
| author | Gao Feng <fgao@ikuai8.com> | 2016-08-28 21:41:22 +0800 |
|---|---|---|
| committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2016-08-30 11:51:08 +0200 |
| commit | 3cb27991aad74299af2cd55a9c12684f5b0c76c6 (patch) | |
| tree | e27c775c6f93761cb96e202a734a085ea27a4a40 | |
| parent | ad66713f5a20034b3b2a0cbc184319b2ede93f11 (diff) | |
| download | linux-3cb27991aad74299af2cd55a9c12684f5b0c76c6.tar.gz linux-3cb27991aad74299af2cd55a9c12684f5b0c76c6.tar.xz | |
netfilter: log_arp: Use ARPHRD_ETHER instead of literal '1'
There is one macro ARPHRD_ETHER which defines the ethernet proto for ARP,
so we could use it instead of the literal number '1'.
Signed-off-by: Gao Feng <fgao@ikuai8.com>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
| -rw-r--r-- | net/ipv4/netfilter/nf_log_arp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ipv4/netfilter/nf_log_arp.c b/net/ipv4/netfilter/nf_log_arp.c index e7ad950cf9ef..cf8f2d4e867a 100644 --- a/net/ipv4/netfilter/nf_log_arp.c +++ b/net/ipv4/netfilter/nf_log_arp.c @@ -62,7 +62,7 @@ static void dump_arp_packet(struct nf_log_buf *m, /* If it's for Ethernet and the lengths are OK, then log the ARP * payload. */ - if (ah->ar_hrd != htons(1) || + if (ah->ar_hrd != htons(ARPHRD_ETHER) || ah->ar_hln != ETH_ALEN || ah->ar_pln != sizeof(__be32)) return; |