diff options
| author |   Linus Lüssing <linus.luessing@c0d3.blue> | 2016-08-07 12:34:18 +0200 |
|---|---|---|
| committer |   Simon Wunderlich <sw@simonwunderlich.de> | 2016-10-30 11:11:34 +0100 |
| commit | eb6915e2eb254fd83dc4472e22ebd6c54bf8ba47 (patch) | |
| tree | 0928ccaa29def7300e79348283445b0f3424880f /net/batman-adv/send.c | |
| parent | batman-adv: Remove unnecessary lockdep in batadv_mcast_mla_list_free (diff) | |
| download | linux-dev-eb6915e2eb254fd83dc4472e22ebd6c54bf8ba47.tar.xz linux-dev-eb6915e2eb254fd83dc4472e22ebd6c54bf8ba47.zip | |
batman-adv: Remove unused skb_reset_mac_header()
During broadcast queueing, the skb_reset_mac_header() sets the skb
to a place invalid for a MAC header, pointing right into the
batman-adv broadcast packet. Luckily, no one seems to actually use
eth_hdr(skb) afterwards until batadv_send_skb_packet() resets the
header to a valid position again.
Therefore removing this unnecessary, weird skb_reset_mac_header()
call.
Signed-off-by: Linus Lüssing <linus.luessing@c0d3.blue>
Signed-off-by: Sven Eckelmann <sven@narfation.org>
Signed-off-by: Simon Wunderlich <sw@simonwunderlich.de>
Diffstat (limited to 'net/batman-adv/send.c')
| -rw-r--r-- | net/batman-adv/send.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/net/batman-adv/send.c b/net/batman-adv/send.c index e1e9136db6e8..be3f6d77f961 100644 --- a/net/batman-adv/send.c +++ b/net/batman-adv/send.c @@ -586,8 +586,6 @@ int batadv_add_bcast_packet_to_list(struct batadv_priv *bat_priv, bcast_packet = (struct batadv_bcast_packet *)newskb->data; bcast_packet->ttl--; - skb_reset_mac_header(newskb); - forw_packet->skb = newskb; INIT_DELAYED_WORK(&forw_packet->delayed_work, |