diff options
author | Johannes Berg <johannes.berg@intel.com> | 2010-05-27 08:41:07 -0400 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2010-06-03 14:10:45 -0400 |
commit | 6a8579d0e62c0eac428184ce45e86bc46677724a (patch) | |
tree | 192d3adc84985598ad6006df43d246adce002f7e /net/mac80211/debugfs_sta.c | |
parent | 2b2c009ecf71f4c66ff8420b63dddbc9737e04e3 (diff) |
mac80211: clean up ieee80211_stop_tx_ba_session
There's no sense in letting anything but internal
mac80211 functions set the initiator to anything
but WLAN_BACK_INITIATOR, since WLAN_BACK_RECIPIENT
is only valid when we have received a frame from
the peer, which we react to directly in mac80211.
The debugfs code I recently added got this wrong
as well.
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'net/mac80211/debugfs_sta.c')
-rw-r--r-- | net/mac80211/debugfs_sta.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/net/mac80211/debugfs_sta.c b/net/mac80211/debugfs_sta.c index e763f1529ddb..9f140612224a 100644 --- a/net/mac80211/debugfs_sta.c +++ b/net/mac80211/debugfs_sta.c | |||
@@ -210,8 +210,7 @@ static ssize_t sta_agg_status_write(struct file *file, const char __user *userbu | |||
210 | if (start) | 210 | if (start) |
211 | ret = ieee80211_start_tx_ba_session(&sta->sta, tid); | 211 | ret = ieee80211_start_tx_ba_session(&sta->sta, tid); |
212 | else | 212 | else |
213 | ret = ieee80211_stop_tx_ba_session(&sta->sta, tid, | 213 | ret = ieee80211_stop_tx_ba_session(&sta->sta, tid); |
214 | WLAN_BACK_RECIPIENT); | ||
215 | } else { | 214 | } else { |
216 | __ieee80211_stop_rx_ba_session(sta, tid, WLAN_BACK_RECIPIENT, 3); | 215 | __ieee80211_stop_rx_ba_session(sta, tid, WLAN_BACK_RECIPIENT, 3); |
217 | ret = 0; | 216 | ret = 0; |