diff options
author | Michal Kazior <michal.kazior@tieto.com> | 2014-01-29 01:56:19 -0500 |
---|---|---|
committer | Johannes Berg <johannes.berg@intel.com> | 2014-02-04 15:58:10 -0500 |
commit | c46a73f39642db4931544a9376338d05aa196df8 (patch) | |
tree | c3c7ce16161e77d2aeaeb6404a6ba22f3ef808d7 /net/mac80211/mlme.c | |
parent | 97518af1260553d2cad71b37a76b597360519e8a (diff) |
mac80211: move csa_active setting in STA CSA
The sdata->vif.csa_active could be left set after,
e.g. channel context constraints check fail in STA
mode leaving the interface in a strange state for
a brief period of time until it is disconnected.
This was harmless but ugly.
Signed-off-by: Michal Kazior <michal.kazior@tieto.com>
Reviewed-by: Luciano Coelho <luciano.coelho@intel.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'net/mac80211/mlme.c')
-rw-r--r-- | net/mac80211/mlme.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/mac80211/mlme.c b/net/mac80211/mlme.c index cadf05905e5a..6c9ebca02394 100644 --- a/net/mac80211/mlme.c +++ b/net/mac80211/mlme.c | |||
@@ -1012,7 +1012,6 @@ ieee80211_sta_process_chanswitch(struct ieee80211_sub_if_data *sdata, | |||
1012 | } | 1012 | } |
1013 | 1013 | ||
1014 | ifmgd->flags |= IEEE80211_STA_CSA_RECEIVED; | 1014 | ifmgd->flags |= IEEE80211_STA_CSA_RECEIVED; |
1015 | sdata->vif.csa_active = true; | ||
1016 | 1015 | ||
1017 | mutex_lock(&local->chanctx_mtx); | 1016 | mutex_lock(&local->chanctx_mtx); |
1018 | if (local->use_chanctx) { | 1017 | if (local->use_chanctx) { |
@@ -1050,6 +1049,7 @@ ieee80211_sta_process_chanswitch(struct ieee80211_sub_if_data *sdata, | |||
1050 | mutex_unlock(&local->chanctx_mtx); | 1049 | mutex_unlock(&local->chanctx_mtx); |
1051 | 1050 | ||
1052 | sdata->csa_chandef = csa_ie.chandef; | 1051 | sdata->csa_chandef = csa_ie.chandef; |
1052 | sdata->vif.csa_active = true; | ||
1053 | 1053 | ||
1054 | if (csa_ie.mode) | 1054 | if (csa_ie.mode) |
1055 | ieee80211_stop_queues_by_reason(&local->hw, | 1055 | ieee80211_stop_queues_by_reason(&local->hw, |