Commit 17f78dd1 authored May 21, 2019 by Jeremy Sowden Committed by Simon Wunderlich May 21, 2019

batman-adv: fix for leaked TVLV handler.



A handler for BATADV_TVLV_ROAM was being registered when the
translation-table was initialized, but not unregistered when the
translation-table was freed.  Unregister it.

Fixes: 122edaa0 ("batman-adv: tvlv - convert roaming adv packet to use tvlv unicast packets")
Reported-by:  <syzbot+d454a826e670502484b8@syzkaller.appspotmail.com>
Signed-off-by: Jeremy Sowden <jeremy@azazel.net>
Signed-off-by: Sven Eckelmann <sven@narfation.org>
Signed-off-by: Simon Wunderlich <sw@simonwunderlich.de>

parent a188339c

net/batman-adv/translation-table.c

+2 −0

Original line number	Diff line number	Diff line
		@@ -3813,6 +3813,8 @@ static void batadv_tt_purge(struct work_struct *work)
		*/
		void batadv_tt_free(struct batadv_priv *bat_priv)
		{
		batadv_tvlv_handler_unregister(bat_priv, BATADV_TVLV_ROAM, 1);

		batadv_tvlv_container_unregister(bat_priv, BATADV_TVLV_TT, 1);
		batadv_tvlv_handler_unregister(bat_priv, BATADV_TVLV_TT, 1);

Admin message