Re: [B.A.T.M.A.N.] [PATCHv2] batman-adv: avoid double free when orig_node initialization fails

On Saturday 15 February 2014 02:17:20 Antonio Quartulli wrote:

In the failure path of the orig_node initialization routine the orig_node->bat_iv.bcast_own field is free'd twice: first in batadv_iv_ogm_orig_get() and then later in batadv_orig_node_free_rcu().

Fix it by removing the kfree in batadv_iv_ogm_orig_get().

Signed-off-by: Antonio Quartulli antonio@meshcoding.com

Change form v1:

• remove kfree from batadv_iv_ogm_orig_get() instead of setting orig_node->bat_iv.bcast_own to NULL

bat_iv_ogm.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-)

Applied in revision d4acda1.

Thanks, Marek