diff options
| author | Emmanuel Grumbach <emmanuel.grumbach@intel.com> | 2016-10-25 10:32:16 +0300 |
|---|---|---|
| committer | Johannes Berg <johannes.berg@intel.com> | 2017-01-11 16:32:58 +0100 |
| commit | d7f842442f766db3f39fc5d166ddcc24bf817056 (patch) | |
| tree | 36d604fe523a57b449d5031c26673d783dbcd9ef /net | |
| parent | c38c39bf7cc04d688291f382469e84ec2a8548a4 (diff) | |
| download | op-kernel-dev-d7f842442f766db3f39fc5d166ddcc24bf817056.zip op-kernel-dev-d7f842442f766db3f39fc5d166ddcc24bf817056.tar.gz | |
mac80211: fix the TID on NDPs sent as EOSP carrier
In the commit below, I forgot to translate the mac80211's
AC to QoS IE order. Moreover, the condition in the if was
wrong. Fix both issues.
This bug would hit only with clients that didn't set all
the ACs as delivery enabled.
Fixes: f438ceb81d4 ("mac80211: uapsd_queues is in QoS IE order")
Signed-off-by: Emmanuel Grumbach <emmanuel.grumbach@intel.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'net')
| -rw-r--r-- | net/mac80211/sta_info.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/mac80211/sta_info.c b/net/mac80211/sta_info.c index b6cfcf0..50c3090 100644 --- a/net/mac80211/sta_info.c +++ b/net/mac80211/sta_info.c @@ -1501,8 +1501,8 @@ ieee80211_sta_ps_deliver_response(struct sta_info *sta, /* This will evaluate to 1, 3, 5 or 7. */ for (ac = IEEE80211_AC_VO; ac < IEEE80211_NUM_ACS; ac++) - if (ignored_acs & BIT(ac)) - continue; + if (!(ignored_acs & ieee80211_ac_to_qos_mask[ac])) + break; tid = 7 - 2 * ac; ieee80211_send_null_response(sta, tid, reason, true, false); |
