batman-adv: Fix bat_v best gw refcnt after netlink dump
authorSven Eckelmann <sven@narfation.org>
Sat, 2 Jun 2018 15:26:35 +0000 (17:26 +0200)
committerSimon Wunderlich <sw@simonwunderlich.de>
Sat, 23 Jun 2018 08:29:33 +0000 (10:29 +0200)
commit9713cb0cf19f1cec6c007e3b37be0697042b6720
tree57b5567dee1aa50a8d5c361d3dffefb1b188b5fa
parentb5685d2687d6612adf5eac519eb7008f74dfd1ec
batman-adv: Fix bat_v best gw refcnt after netlink dump

A reference for 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_v_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: b71bb6f924fe ("batman-adv: add B.A.T.M.A.N. V bat_gw_dump implementations")
Signed-off-by: Sven Eckelmann <sven@narfation.org>
Acked-by: Marek Lindner <mareklindner@neomailbox.ch>
Signed-off-by: Simon Wunderlich <sw@simonwunderlich.de>
net/batman-adv/bat_v.c