On Friday 12 February 2016 11:35:35 Simon Wunderlich wrote:

From: Simon Wunderlich simon.wunderlich@open-mesh.com

My latest restructure attempt of the BATMAN v forward function introduced a regression, causing kernel crashes when an OGM is forwarded. This patch fixes it.

Fixes: 30c96bc787 ("batman-adv: move and restructure batadv_v_ogm_forward") Reported-by: Sven Eckelmann sven.eckelmann@open-mesh.com Signed-off-by: Simon Wunderlich simon.wunderlich@open-mesh.com

Thanks for the fast update. It fixes the problem in my setup

Tested-by: Sven Eckelmann sven.eckelmann@open-mesh.com

---

net/batman-adv/bat_v_ogm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/net/batman-adv/bat_v_ogm.c b/net/batman-adv/bat_v_ogm.c index 1b2399e..4155fa5 100644 --- a/net/batman-adv/bat_v_ogm.c +++ b/net/batman-adv/bat_v_ogm.c @@ -309,7 +309,7 @@ static void batadv_v_ogm_forward(struct batadv_priv *bat_priv, u16 tvlv_len;

/* only forward for specific interfaces, not for the default one. */

• if (if_outgoing != BATADV_IF_DEFAULT)

• if (if_outgoing == BATADV_IF_DEFAULT) goto out;

  orig_ifinfo = batadv_orig_ifinfo_new(orig_node, if_outgoing);