Re: [B.A.T.M.A.N.] [PATCH] batman-adv: Fix check of retrieved orig_gw in batadv_v_gw_is_eligible

2017-11-29 Thread Antonio Quartulli
On 29/11/17 17:50, Sven Eckelmann wrote: > The batadv_v_gw_is_eligible function already assumes that orig_node is not > NULL. But batadv_gw_node_get may have failed to find the originator. It > must therefore be checked whether the batadv_gw_node_get failed and not > whether orig_node is NULL to

[B.A.T.M.A.N.] [PATCH] batman-adv: Fix check of retrieved orig_gw in batadv_v_gw_is_eligible

2017-11-29 Thread Sven Eckelmann
The batadv_v_gw_is_eligible function already assumes that orig_node is not NULL. But batadv_gw_node_get may have failed to find the originator. It must therefore be checked whether the batadv_gw_node_get failed and not whether orig_node is NULL to detect this error. Fixes: 80b2d47be2c7

[B.A.T.M.A.N.] [PATCH] batman-adv: Always initialize fragment header priority

2017-11-29 Thread Sven Eckelmann
From: Sven Eckelmann The batman-adv unuicast fragment header contains 3 bits for the priority of the packet. These bits will be initialized when the skb->priority contains a value between 256 and 263. But otherwise, the uninitialized bits from the stack will be