diff options
author | Johannes Berg <johannes.berg@intel.com> | 2017-11-13 11:33:55 +0100 |
---|---|---|
committer | Johannes Berg <johannes.berg@intel.com> | 2017-11-20 17:01:31 +0100 |
commit | 33ddd81e2bd5d9970b9f01ab383ba45035fa41ee (patch) | |
tree | 8ac9805306fbd178c65b8cf42eef0bc1f19118f2 /firmware | |
parent | 67bd52386125ce1159c0581cbcd2740addf33cd4 (diff) |
mac80211: properly free requested-but-not-started TX agg sessions
When deleting a station or otherwise tearing down all aggregation
sessions, make sure to delete requested but not yet started ones,
to avoid the following scenario:
* session is requested, added to tid_start_tx[]
* ieee80211_ba_session_work() runs, gets past BLOCK_BA check
* ieee80211_sta_tear_down_BA_sessions() runs, locks &sta->ampdu_mlme.mtx,
e.g. while deleting the station - deleting all active sessions
* ieee80211_ba_session_work() continues since tear down flushes it, and
calls ieee80211_tx_ba_session_handle_start() for the new session, arms
the timer for it
* station deletion continues to __cleanup_single_sta() and frees the
session struct, while the timer is armed
Reported-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'firmware')
0 files changed, 0 insertions, 0 deletions