Repository : ssh://git@open-mesh.org/batman-adv

On branch : next

commit 846815f0af43bdc2c58cb5e1b64c158edefda16e Author: Antonio Quartulli ordex@autistici.org Date: Mon Nov 5 16:34:34 2012 +0100

batman-adv: fix ROAM flag de-selection

In tt_global_add() the ROAM flag is improperly unset. This makes the next check fail and so preventing emptying the old originator_list for the involved tt_global_entry

This was introduced by ("batman-adv: roaming handling mechanism redesign")

Signed-off-by: Antonio Quartulli ordex@autistici.org Reported-by: Mike Burmeister-Brown mike@open-mesh.com Signed-off-by: Marek Lindner lindner_marek@yahoo.de

846815f0af43bdc2c58cb5e1b64c158edefda16e translation-table.c | 8 -------- 1 file changed, 8 deletions(-)

diff --git a/translation-table.c b/translation-table.c index c731050..0bdf980 100644 --- a/translation-table.c +++ b/translation-table.c @@ -860,14 +860,6 @@ int batadv_tt_global_add(struct batadv_priv *bat_priv, */ common->flags &= ~BATADV_TT_CLIENT_TEMP;

- /* if this is a normal add, possibly unset the ROAM flag. This - * flag could have been set before because the client was - * roaming, but now that the node got the ADD event, the flag - * can be unset - */ - if (!(flags & BATADV_TT_CLIENT_ROAM)) - common->flags &= ~BATADV_TT_CLIENT_ROAM; - /* If there is the BATADV_TT_CLIENT_ROAM flag set, there is only * one originator left in the list and we previously received a * delete + roaming change for this originator.