diff options
| author | Colin Ian King <[email protected]> | 2017-01-13 18:48:20 +0000 |
|---|---|---|
| committer | David S. Miller <[email protected]> | 2017-01-16 13:48:48 -0500 |
| commit | 57b68ec2a7be9a7e9f8999850f8ee5baa49023e4 (patch) | |
| tree | 14d141beaf15ea3c11936e3eaa4211148c4d4f0a /tools/perf/scripts/python/bin | |
| parent | e89df813174d81abee8be8cabc047d69ef78c26d (diff) | |
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 <[email protected]>
Acked-by: Eric Dumazet <[email protected]>
Signed-off-by: David S. Miller <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/bin')
0 files changed, 0 insertions, 0 deletions