mac80211: ftm responder: remove pointless defensive coding

The pointer and corresponding length is always set in pairs
in cfg80211, so no need to have this strange defensive check
that also confuses static checkers. Clean it up.

Signed-off-by: Johannes Berg <johannes.berg@intel.com>

diff --git a/net/mac80211/cfg.c b/net/mac80211/cfg.c
index cf8f946ae724c5233bf977e6c87443e8ed93111e..567c63ff830f4ab1c53ce88dcf99708b5e6e1968 100644 (file)
--- a/net/mac80211/cfg.c
+++ b/net/mac80211/cfg.c
@@ -800,8 +800,8 @@ static int ieee80211_set_ftm_responder_params(
        u8 *pos;
        int len;
 
-       if ((!lci || !lci_len) && (!civicloc || !civicloc_len))
-               return 1;
+       if (!lci_len && !civicloc_len)
+               return 0;
 
        bss_conf = &sdata->vif.bss_conf;
        old = bss_conf->ftmr_params;