Commit 9b7a86f3 authored by Johannes Berg's avatar Johannes Berg

mac80211: fix handling TIM IE when stations disconnect

When a station disconnects with frames still pending, we clear
the TIM bit, but too late - it's only cleared when the station
is already removed from the driver, and thus the driver can get
confused (and hwsim will loudly complain.)

Fix this by clearing the TIM bit earlier, when the station has
been unlinked but not removed from the driver yet. To do this,
refactor the TIM recalculation to in that case ignore traffic
and simply assume no pending traffic - this is correct for the
disconnected station even though the frames haven't been freed
yet at that point.

This patch isn't needed for current drivers though as they don't
check the station argument to the set_tim() operation and thus
don't really run into the possible confusion.
Reported-by: default avatarJouni Malinen <j@w1.fi>
Signed-off-by: default avatarJohannes Berg <johannes.berg@intel.com>
parent a809ca5e
...@@ -116,7 +116,6 @@ static void __cleanup_single_sta(struct sta_info *sta) ...@@ -116,7 +116,6 @@ static void __cleanup_single_sta(struct sta_info *sta)
clear_sta_flag(sta, WLAN_STA_PS_DELIVER); clear_sta_flag(sta, WLAN_STA_PS_DELIVER);
atomic_dec(&ps->num_sta_ps); atomic_dec(&ps->num_sta_ps);
sta_info_recalc_tim(sta);
} }
for (ac = 0; ac < IEEE80211_NUM_ACS; ac++) { for (ac = 0; ac < IEEE80211_NUM_ACS; ac++) {
...@@ -625,7 +624,7 @@ static unsigned long ieee80211_tids_for_ac(int ac) ...@@ -625,7 +624,7 @@ static unsigned long ieee80211_tids_for_ac(int ac)
} }
} }
void sta_info_recalc_tim(struct sta_info *sta) static void __sta_info_recalc_tim(struct sta_info *sta, bool ignore_pending)
{ {
struct ieee80211_local *local = sta->local; struct ieee80211_local *local = sta->local;
struct ps_data *ps; struct ps_data *ps;
...@@ -667,6 +666,9 @@ void sta_info_recalc_tim(struct sta_info *sta) ...@@ -667,6 +666,9 @@ void sta_info_recalc_tim(struct sta_info *sta)
if (ignore_for_tim == BIT(IEEE80211_NUM_ACS) - 1) if (ignore_for_tim == BIT(IEEE80211_NUM_ACS) - 1)
ignore_for_tim = 0; ignore_for_tim = 0;
if (ignore_pending)
ignore_for_tim = BIT(IEEE80211_NUM_ACS) - 1;
for (ac = 0; ac < IEEE80211_NUM_ACS; ac++) { for (ac = 0; ac < IEEE80211_NUM_ACS; ac++) {
unsigned long tids; unsigned long tids;
...@@ -695,7 +697,7 @@ void sta_info_recalc_tim(struct sta_info *sta) ...@@ -695,7 +697,7 @@ void sta_info_recalc_tim(struct sta_info *sta)
else else
__bss_tim_clear(ps->tim, id); __bss_tim_clear(ps->tim, id);
if (local->ops->set_tim) { if (local->ops->set_tim && !WARN_ON(sta->dead)) {
local->tim_in_locked_section = true; local->tim_in_locked_section = true;
drv_set_tim(local, &sta->sta, indicate_tim); drv_set_tim(local, &sta->sta, indicate_tim);
local->tim_in_locked_section = false; local->tim_in_locked_section = false;
...@@ -705,6 +707,11 @@ void sta_info_recalc_tim(struct sta_info *sta) ...@@ -705,6 +707,11 @@ void sta_info_recalc_tim(struct sta_info *sta)
spin_unlock_bh(&local->tim_lock); spin_unlock_bh(&local->tim_lock);
} }
void sta_info_recalc_tim(struct sta_info *sta)
{
__sta_info_recalc_tim(sta, false);
}
static bool sta_info_buffer_expired(struct sta_info *sta, struct sk_buff *skb) static bool sta_info_buffer_expired(struct sta_info *sta, struct sk_buff *skb)
{ {
struct ieee80211_tx_info *info; struct ieee80211_tx_info *info;
...@@ -888,6 +895,9 @@ static void __sta_info_destroy_part2(struct sta_info *sta) ...@@ -888,6 +895,9 @@ static void __sta_info_destroy_part2(struct sta_info *sta)
/* now keys can no longer be reached */ /* now keys can no longer be reached */
ieee80211_free_sta_keys(local, sta); ieee80211_free_sta_keys(local, sta);
/* disable TIM bit - last chance to tell driver */
__sta_info_recalc_tim(sta, true);
sta->dead = true; sta->dead = true;
local->num_sta--; local->num_sta--;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment