Commit a1a745ef authored by Andreas Pape's avatar Andreas Pape Committed by Simon Wunderlich

batman-adv: fix memory leak when dropping packet from other gateway

The skb must be released in the receive handler since b91a2543
("batman-adv: Consume skb in receive handlers"). Just returning NET_RX_DROP
will no longer automatically free the memory. This results in memory leaks
when unicast packets from other backbones must be dropped because they
share a common backbone.

Fixes: 9e794b6b ("batman-adv: drop unicast packets from other backbone gw")
Signed-off-by: default avatarAndreas Pape <apape@phoenixcontact.com>
[sven@narfation.org: adjust commit message]
Signed-off-by: default avatarSven Eckelmann <sven@narfation.org>
Signed-off-by: default avatarSimon Wunderlich <sw@simonwunderlich.de>
parent 36d4d68c
...@@ -987,7 +987,7 @@ int batadv_recv_unicast_packet(struct sk_buff *skb, ...@@ -987,7 +987,7 @@ int batadv_recv_unicast_packet(struct sk_buff *skb,
batadv_dbg(BATADV_DBG_BLA, bat_priv, batadv_dbg(BATADV_DBG_BLA, bat_priv,
"recv_unicast_packet(): Dropped unicast pkt received from another backbone gw %pM.\n", "recv_unicast_packet(): Dropped unicast pkt received from another backbone gw %pM.\n",
orig_addr_gw); orig_addr_gw);
return NET_RX_DROP; goto free_skb;
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment