Commit 1adf643050df1289c36dd158baa5d4cc2fa593ae

Authored by Simon Wunderlich
Committed by Greg Kroah-Hartman
1 parent c712c1f79c

batman-adv: fix neigh_ifinfo imbalance

[ Upstream commit c1e517fbbcdb13f50662af4edc11c3251fe44f86 ]

The neigh_ifinfo object must be freed if it has been used in
batadv_iv_ogm_process_per_outif().

This is a regression introduced by
89652331c00f43574515059ecbf262d26d885717
("batman-adv: split tq information in neigh_node struct")

Reported-by: Antonio Quartulli <antonio@open-mesh.com>
Signed-off-by: Simon Wunderlich <simon@open-mesh.com>
Signed-off-by: Marek Lindner <mareklindner@neomailbox.ch>
Signed-off-by: Antonio Quartulli <antonio@meshcoding.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

Showing 1 changed file with 2 additions and 0 deletions Side-by-side Diff

net/batman-adv/bat_iv_ogm.c
... ... @@ -1545,6 +1545,8 @@
1545 1545 if ((orig_neigh_node) && (!is_single_hop_neigh))
1546 1546 batadv_orig_node_free_ref(orig_neigh_node);
1547 1547 out:
  1548 + if (router_ifinfo)
  1549 + batadv_neigh_ifinfo_free_ref(router_ifinfo);
1548 1550 if (router)
1549 1551 batadv_neigh_node_free_ref(router);
1550 1552 if (router_router)