wifi: mac80211: remove erroneous sband/link validation
authorJohannes Berg <johannes.berg@intel.com>
Thu, 21 Jul 2022 22:07:29 +0000 (00:07 +0200)
committerJohannes Berg <johannes.berg@intel.com>
Fri, 22 Jul 2022 12:28:45 +0000 (14:28 +0200)
In sta_apply_parameters(), we really no longer need to
check that the link or sband exists, in fact, that's
harmful if link 0 doesn't exist, since then this will
fail.

Just remove this check, it was added for validation of
the sband where used, but it's not used here, it's now
only used in sta_link_apply_parameters() which has an
own lookup and check.

Signed-off-by: Johannes Berg <johannes.berg@intel.com>
net/mac80211/cfg.c

index f519d9c..a4f6971 100644 (file)
@@ -1684,21 +1684,9 @@ static int sta_apply_parameters(struct ieee80211_local *local,
                                struct sta_info *sta,
                                struct station_parameters *params)
 {
-       int ret = 0;
-       struct ieee80211_supported_band *sband;
        struct ieee80211_sub_if_data *sdata = sta->sdata;
-       u32 link_id = params->link_sta_params.link_id < 0 ?
-                     0 : params->link_sta_params.link_id;
-       struct ieee80211_link_data *link;
        u32 mask, set;
-
-       link = sdata_dereference(sdata->link[link_id], sdata);
-       if (!link)
-               return -ENOLINK;
-
-       sband = ieee80211_get_link_sband(link);
-       if (!sband)
-               return -EINVAL;
+       int ret = 0;
 
        mask = params->sta_flags_mask;
        set = params->sta_flags_set;