Skip to content

Commit

Permalink
mac80211: use correct station flags lock
Browse files Browse the repository at this point in the history
This code is modifying the station flags, and
as such should hold the flags lock so it can
do so atomically vs. other flags modifications
and readers. This issue was introduced when
this code was added in eccb8e8, as it used
the wrong lock (thus not fixing the race that
was previously documented in a comment.)

Cc: stable@kernel.org [2.6.31+]
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
  • Loading branch information
Johannes Berg authored and John W. Linville committed Sep 16, 2010
1 parent 3905751 commit f5521b1
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions net/mac80211/cfg.c
Original file line number Diff line number Diff line change
Expand Up @@ -577,6 +577,7 @@ static void sta_apply_parameters(struct ieee80211_local *local,
struct sta_info *sta,
struct station_parameters *params)
{
unsigned long flags;
u32 rates;
int i, j;
struct ieee80211_supported_band *sband;
Expand All @@ -585,7 +586,7 @@ static void sta_apply_parameters(struct ieee80211_local *local,

sband = local->hw.wiphy->bands[local->oper_channel->band];

spin_lock_bh(&sta->lock);
spin_lock_irqsave(&sta->flaglock, flags);
mask = params->sta_flags_mask;
set = params->sta_flags_set;

Expand All @@ -612,7 +613,7 @@ static void sta_apply_parameters(struct ieee80211_local *local,
if (set & BIT(NL80211_STA_FLAG_MFP))
sta->flags |= WLAN_STA_MFP;
}
spin_unlock_bh(&sta->lock);
spin_unlock_irqrestore(&sta->flaglock, flags);

/*
* cfg80211 validates this (1-2007) and allows setting the AID
Expand Down

0 comments on commit f5521b1

Please sign in to comment.