batman-adv: TT_CLIENT_PENDING is never set in struct tt_global_entry

Actually the TT_CLIENT_PENDING flag is never set in the tt_global_entry
structure, therefore this code is useless and can be removed.

Reported-by: Simon Wunderlich <siwu@hrz.tu-chemnitz.de>
Signed-off-by: Antonio Quartulli <ordex@autistici.org>
Signed-off-by: Marek Lindner <lindner_marek@yahoo.de>

1 files changed, 1 insertions, 8 deletions

diff --git a/net/batman-adv/translation-table.c b/net/batman-adv/translation-table.c
index 1ff7fbebf921..9a6f315b7d44 100644
--- a/net/batman-adv/translation-table.c
+++ b/net/batman-adv/translation-table.c
@@ -603,7 +603,7 @@ int tt_global_seq_print_text(struct seq_file *seq, void *offset)
                                                        struct tt_global_entry,
                                                        common);
                         seq_printf(seq, " * %pM  (%3u) via %pM     (%3u)   "
-                                        "[%c%c%c]\n",
+                                        "[%c%c]\n",
                                         tt_global_entry->common.addr,
                                         tt_global_entry->ttvn,
                                         tt_global_entry->orig_node->orig,
@@ -613,8 +613,6 @@ int tt_global_seq_print_text(struct seq_file *seq, void *offset)
                                         (tt_global_entry->common.flags &
                                          TT_CLIENT_ROAM ? 'R' : '.'),
                                         (tt_global_entry->common.flags &
-                                         TT_CLIENT_PENDING ? 'X' : '.'),
-                                        (tt_global_entry->common.flags &
                                          TT_CLIENT_WIFI ? 'W' : '.'));
                 }
                 rcu_read_unlock();
@@ -838,11 +836,6 @@ struct orig_node *transtable_search(struct bat_priv *bat_priv,
         if (!atomic_inc_not_zero(&tt_global_entry->orig_node->refcount))
                 goto out;
 
-        /* A global client marked as PENDING has already moved from that
-         * originator */
-        if (tt_global_entry->common.flags & TT_CLIENT_PENDING)
-                goto out;
-
         orig_node = tt_global_entry->orig_node;
 
 out: