diff options
| author |   Johannes Berg <johannes.berg@intel.com> | 2016-09-14 11:38:31 +0200 |
|---|---|---|
| committer | Johannes Berg <johannes.berg@intel.com> | 2016-09-15 10:08:52 +0200 |
| commit | 85d5313ed717ad60769491c7c072d23bc0a68e7a (patch) | |
| tree | 4f39837717938d9cbf341704d0c8d06113b6b3f2 /net/mac80211/agg-tx.c | |
| parent | mac80211: check skb_linearize() return value (diff) | |
| download | linux-dev-85d5313ed717ad60769491c7c072d23bc0a68e7a.tar.xz linux-dev-85d5313ed717ad60769491c7c072d23bc0a68e7a.zip | |
mac80211: reject TSPEC TIDs (TSIDs) for aggregation
Since mac80211 doesn't currently support TSIDs 8-15 which can
only be used after QoS TSPEC negotiation (and not even after
WMM negotiation), reject attempts to set up aggregation
sessions for them, which might confuse drivers. In mac80211
we do correctly handle that, but the TSIDs should never get
used anyway, and drivers might not be able to handle it.
Cc: stable@vger.kernel.org
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'net/mac80211/agg-tx.c')
| -rw-r--r-- | net/mac80211/agg-tx.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/net/mac80211/agg-tx.c b/net/mac80211/agg-tx.c index 5650c46bf91a..45319cc01121 100644 --- a/net/mac80211/agg-tx.c +++ b/net/mac80211/agg-tx.c @@ -584,6 +584,9 @@ int ieee80211_start_tx_ba_session(struct ieee80211_sta *pubsta, u16 tid, ieee80211_hw_check(&local->hw, TX_AMPDU_SETUP_IN_HW)) return -EINVAL; + if (WARN_ON(tid >= IEEE80211_FIRST_TSPEC_TSID)) + return -EINVAL; + ht_dbg(sdata, "Open BA session requested for %pM tid %u\n", pubsta->addr, tid); |