diff options
| author | Johannes Berg <johannes.berg@intel.com> | 2014-03-04 11:43:28 +0100 |
|---|---|---|
| committer | Jiri Slaby <jslaby@suse.cz> | 2014-05-15 09:56:03 +0200 |
| commit | 243677b1125ce3f460e45d887127072c6d323579 (patch) | |
| tree | 3acaa169292910abbb0897dc0b9cf5af198f4296 | |
| parent | 02f079bf44c85322a0383d6e0e10fe5752f61595 (diff) | |
mac80211: fix potential use-after-free
commit d2722f8b87fb172ff2f31d3a2816b31d58678d40 upstream.
The bss struct might be freed in ieee80211_rx_bss_put(),
so we shouldn't use it afterwards.
Fixes: 817cee7675237 ("mac80211: track AP's beacon rate and give it to the driver")
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
| -rw-r--r-- | net/mac80211/mlme.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/mac80211/mlme.c b/net/mac80211/mlme.c index 8d7f4abe65ba..2af1c38374a4 100644 --- a/net/mac80211/mlme.c +++ b/net/mac80211/mlme.c @@ -2881,8 +2881,8 @@ static void ieee80211_rx_bss_info(struct ieee80211_sub_if_data *sdata, bss = ieee80211_bss_info_update(local, rx_status, mgmt, len, elems, channel); if (bss) { - ieee80211_rx_bss_put(local, bss); sdata->vif.bss_conf.beacon_rate = bss->beacon_rate; + ieee80211_rx_bss_put(local, bss); } } |