diff options
| author | Colin Ian King <colin.king@canonical.com> | 2017-01-13 18:48:20 +0000 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2017-01-16 13:48:48 -0500 |
| commit | 57b68ec2a7be9a7e9f8999850f8ee5baa49023e4 (patch) | |
| tree | 14d141beaf15ea3c11936e3eaa4211148c4d4f0a /net/sctp | |
| parent | e89df813174d81abee8be8cabc047d69ef78c26d (diff) | |
| download | linux-57b68ec2a7be9a7e9f8999850f8ee5baa49023e4.tar.bz2 | |
flow dissector: check if arp_eth is null rather than arp
arp is being checked instead of arp_eth to see if the call to
__skb_header_pointer failed. Fix this by checking arp_eth is
null instead of arp. Also fix to use length hlen rather than
hlen - sizeof(_arp); thanks to Eric Dumazet for spotting
this latter issue.
CoverityScan CID#1396428 ("Logically dead code") on 2nd
arp comparison (which should be arp_eth instead).
Fixes: commit 55733350e5e8b70c5 ("flow disector: ARP support")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Acked-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sctp')
0 files changed, 0 insertions, 0 deletions