[B.A.T.M.A.N.] [PATCH 3/3] batman-adv: Less function calls in batadv_is_ap_isolated() after error detection

[SF Markus Elfring]

From: Markus Elfring <elfr...@users.sourceforge.net>
Date: Tue, 3 Nov 2015 21:10:51 +0100

The variables "tt_local_entry" and "tt_global_entry" were eventually checked
again despite of a corresponding null pointer test before.
Let us avoid this double check by reordering a function call sequence
and the better selection of jump targets.

Signed-off-by: Markus Elfring <elfr...@users.sourceforge.net>
---
 net/batman-adv/translation-table.c | 21 +++++++++------------
 1 file changed, 9 insertions(+), 12 deletions(-)

diff --git a/net/batman-adv/translation-table.c 
b/net/batman-adv/translation-table.c
index 965a004..3ac32d9 100644
--- a/net/batman-adv/translation-table.c
+++ b/net/batman-adv/translation-table.c
@@ -3323,27 +3323,24 @@ bool batadv_is_ap_isolated(struct batadv_priv 
*bat_priv, u8 *src, u8 *dst,
                return false;
 
        if (!atomic_read(&vlan->ap_isolation))
-               goto out;
+               goto vlan_free;
 
        tt_local_entry = batadv_tt_local_hash_find(bat_priv, dst, vid);
        if (!tt_local_entry)
-               goto out;
+               goto vlan_free;
 
        tt_global_entry = batadv_tt_global_hash_find(bat_priv, src, vid);
        if (!tt_global_entry)
-               goto out;
+               goto local_entry_free;
 
-       if (!_batadv_is_ap_isolated(tt_local_entry, tt_global_entry))
-               goto out;
-
-       ret = true;
+       if (_batadv_is_ap_isolated(tt_local_entry, tt_global_entry))
+               ret = true;
 
-out:
+       batadv_tt_global_entry_free_ref(tt_global_entry);
+local_entry_free:
+       batadv_tt_local_entry_free_ref(tt_local_entry);
+vlan_free:
        batadv_softif_vlan_free_ref(vlan);
-       if (tt_global_entry)
-               batadv_tt_global_entry_free_ref(tt_global_entry);
-       if (tt_local_entry)
-               batadv_tt_local_entry_free_ref(tt_local_entry);
        return ret;
 }
 
-- 
2.6.2