diff options
author | Johannes Berg <johannes.berg@intel.com> | 2012-08-01 10:13:02 -0400 |
---|---|---|
committer | Johannes Berg <johannes.berg@intel.com> | 2012-08-20 07:57:50 -0400 |
commit | 5bc1420b11903e9f8c470d3b33061b8de0c5c005 (patch) | |
tree | 5fa942edea59ad047aa58f9d84c6259cd3b4f314 /net/mac80211/mlme.c | |
parent | 3049000b97bbfc90aa9ba413eadc4007e5bce2e0 (diff) |
mac80211: check size of channel switch IE when parsing
The channel switch IE has a fixed size, so we can
discard it in parsing if it's not the right size
and use the right struct pointer.
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'net/mac80211/mlme.c')
-rw-r--r-- | net/mac80211/mlme.c | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/net/mac80211/mlme.c b/net/mac80211/mlme.c index 9e61fe127a33..b9cb8dbe34d9 100644 --- a/net/mac80211/mlme.c +++ b/net/mac80211/mlme.c | |||
@@ -2267,14 +2267,10 @@ static void ieee80211_rx_bss_info(struct ieee80211_sub_if_data *sdata, | |||
2267 | mutex_unlock(&local->iflist_mtx); | 2267 | mutex_unlock(&local->iflist_mtx); |
2268 | } | 2268 | } |
2269 | 2269 | ||
2270 | if (elems->ch_switch_elem && (elems->ch_switch_elem_len == 3) && | 2270 | if (elems->ch_switch_ie && |
2271 | (memcmp(mgmt->bssid, sdata->u.mgd.associated->bssid, | 2271 | memcmp(mgmt->bssid, sdata->u.mgd.associated->bssid, ETH_ALEN) == 0) |
2272 | ETH_ALEN) == 0)) { | 2272 | ieee80211_sta_process_chanswitch(sdata, elems->ch_switch_ie, |
2273 | struct ieee80211_channel_sw_ie *sw_elem = | ||
2274 | (struct ieee80211_channel_sw_ie *)elems->ch_switch_elem; | ||
2275 | ieee80211_sta_process_chanswitch(sdata, sw_elem, | ||
2276 | bss, rx_status->mactime); | 2273 | bss, rx_status->mactime); |
2277 | } | ||
2278 | } | 2274 | } |
2279 | 2275 | ||
2280 | 2276 | ||