diff mbox

batman-adv: refactor tt_global_del() to avoid misalignment

Message ID 1324557222-7005-1-git-send-email-lindner_marek@yahoo.de (mailing list archive)
State Accepted, archived
Commit e2b9917bb63b44deb41b2a2bdb4033dc6929d730
Headers

Commit Message

Marek Lindner Dec. 22, 2011, 12:33 p.m. UTC 
  From: Sven Eckelmann <sven@narfation.org>

Signed-off-by: Sven Eckelmann <sven@narfation.org>
---
 translation-table.c |   42 ++++++++++++++++++++++--------------------
 1 files changed, 22 insertions(+), 20 deletions(-)

Comments

Antonio Quartulli Dec. 22, 2011, 2:42 p.m. UTC | #1 
On Thu, Dec 22, 2011 at 08:33:42PM +0800, Marek Lindner wrote:
> From: Sven Eckelmann <sven@narfation.org>
> 
> Signed-off-by: Sven Eckelmann <sven@narfation.org>

Since this is a part of the code I wrote, I gave it a look in order to be
sure that nothing has been messed up.

Acked-by: Antonio Quartulli <ordex@autistici.org>
Marek Lindner Dec. 24, 2011, 1:59 p.m. UTC | #2 
On Thursday, December 22, 2011 22:42:24 Antonio Quartulli wrote:
> On Thu, Dec 22, 2011 at 08:33:42PM +0800, Marek Lindner wrote:
> > From: Sven Eckelmann <sven@narfation.org>
> >
> > 
> >
> > Signed-off-by: Sven Eckelmann <sven@narfation.org>
> 
> Since this is a part of the code I wrote, I gave it a look in order to be
> sure that nothing has been messed up.
> 
> Acked-by: Antonio Quartulli <ordex@autistici.org>


Applied in revision e2b9917.

Thanks,
Marek
diff mbox

Patch

diff --git a/translation-table.c b/translation-table.c
index d542b9d..a1be7ef 100644
--- a/translation-table.c
+++ b/translation-table.c
@@ -657,29 +657,31 @@  void tt_global_del(struct bat_priv *bat_priv,
 	struct tt_local_entry *tt_local_entry = NULL;
 
 	tt_global_entry = tt_global_hash_find(bat_priv, addr);
-	if (!tt_global_entry)
+	if (!tt_global_entry || tt_global_entry->orig_node != orig_node)
 		goto out;
 
-	if (tt_global_entry->orig_node == orig_node) {
-		if (roaming) {
-			/* if we are deleting a global entry due to a roam
-			 * event, there are two possibilities:
-			 * 1) the client roamed from node A to node B => we mark
-			 *    it with TT_CLIENT_ROAM, we start a timer and we
-			 *    wait for node B to claim it. In case of timeout
-			 *    the entry is purged.
-			 * 2) the client roamed to us => we can directly delete
-			 *    the global entry, since it is useless now. */
-			tt_local_entry = tt_local_hash_find(bat_priv,
-						tt_global_entry->common.addr);
-			if (!tt_local_entry) {
-				tt_global_entry->common.flags |= TT_CLIENT_ROAM;
-				tt_global_entry->roam_at = jiffies;
-				goto out;
-			}
-		}
-		_tt_global_del(bat_priv, tt_global_entry, message);
+	if (!roaming)
+		goto out_del;
+
+	/* if we are deleting a global entry due to a roam
+	 * event, there are two possibilities:
+	 * 1) the client roamed from node A to node B => we mark
+	 *    it with TT_CLIENT_ROAM, we start a timer and we
+	 *    wait for node B to claim it. In case of timeout
+	 *    the entry is purged.
+	 * 2) the client roamed to us => we can directly delete
+	 *    the global entry, since it is useless now. */
+	tt_local_entry = tt_local_hash_find(bat_priv,
+					    tt_global_entry->common.addr);
+	if (!tt_local_entry) {
+		tt_global_entry->common.flags |= TT_CLIENT_ROAM;
+		tt_global_entry->roam_at = jiffies;
+		goto out;
 	}
+
+out_del:
+	_tt_global_del(bat_priv, tt_global_entry, message);
+
 out:
 	if (tt_global_entry)
 		tt_global_entry_free_ref(tt_global_entry);