]> asedeno.scripts.mit.edu Git - linux.git/commit
batman-adv: Adjust name for batadv_dat_send_data
authorSven Eckelmann <sven@narfation.org>
Sun, 17 Mar 2019 09:50:50 +0000 (10:50 +0100)
committerSimon Wunderlich <sw@simonwunderlich.de>
Mon, 25 Mar 2019 09:01:13 +0000 (10:01 +0100)
commitc2d8b9a6c17a3848136b3eb31f26d3c5880acd89
tree8a35628b86144762f834932707387765817dd785
parentcedb0dbbb2b0811e58c999a0eb57739e29424352
batman-adv: Adjust name for batadv_dat_send_data

The send functions in batman-adv are expected to consume the skb when
either the data is queued up for the underlying driver or when some
precondition failed. batadv_dat_send_data didn't do this and instead
created a copy of the skb, modified it and queued the copy up for
transmission. The caller has to take care that the skb is handled correctly
(for example free'd) when batadv_dat_send_data returns.

This unclear behavior already lead to memory leaks in the recent past.
Renaming the function to batadv_dat_forward_data should make it easier to
identify that the data is forwarded but the skb is not actually
send+consumed.

Signed-off-by: Sven Eckelmann <sven@narfation.org>
Signed-off-by: Simon Wunderlich <sw@simonwunderlich.de>
net/batman-adv/distributed-arp-table.c