On Samstag, 2. Juni 2018 17:26:34 CEST Sven Eckelmann wrote:
A reference to the best gateway is taken when the list of gateways in the mesh is sent via netlink. This is necessary to check whether the currently dumped entry is the currently selected gateway or not. This information is then transferred as flag BATADV_ATTR_FLAG_BEST.
After the comparison of the current entry is done, batadv_iv_gw_dump_entry() has to decrease the reference counter again. Otherwise the reference will be held and thus prevents a proper shutdown of the batman-adv interfaces (and some of the interfaces enslaved in it).
Fixes: fa3228924152 ("batman-adv: add B.A.T.M.A.N. IV bat_gw_dump implementations") Reported-by: Andreas Ziegler github@andreas-ziegler.de Signed-off-by: Sven Eckelmann sven@narfation.org
Cc: Andreas Ziegler github@andreas-ziegler.de
Added as 46360d203c62 [1]
Kind regards, Sven
[1] https://git.open-mesh.org/batman-adv.git/commit/46360d203c627e71a27d1f8f551c...