mirror of
https://github.com/vanhoefm/fragattacks.git
synced 2024-11-29 02:38:22 -05:00
nl80211: Report disassociated STA / lost peer for the correct BSS
We shouldn't use drv->ctx as it always points to the first BSS. When using FullMAC driver with multi-BSS support it resulted in incorrect treating nl80211 events. I noticed with with brcmfmac and BCM43602. Before my change I was getting "disassociated" on a wrong interface: wlan0-1: STA 78:d6:f0:00:11:22 IEEE 802.11: associated wlan0-1: STA 78:d6:f0:00:11:22 WPA: pairwise key handshake completed (RSN) wlan0: STA 78:d6:f0:00:11:22 IEEE 802.11: disassociated With this patch it works as expected: wlan0-1: STA 78:d6:f0:00:11:22 IEEE 802.11: associated wlan0-1: STA 78:d6:f0:00:11:22 WPA: pairwise key handshake completed (RSN) wlan0-1: STA 78:d6:f0:00:11:22 IEEE 802.11: disassociated This doesn't apply to hostapd dealing with SoftMAC drivers when handling AP SME & MLME is done it hostapd not the firmware. Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
This commit is contained in:
parent
38af0423cb
commit
4653ceb750
@ -1181,6 +1181,7 @@ static void nl80211_new_station_event(struct wpa_driver_nl80211_data *drv,
|
||||
|
||||
|
||||
static void nl80211_del_station_event(struct wpa_driver_nl80211_data *drv,
|
||||
struct i802_bss *bss,
|
||||
struct nlattr **tb)
|
||||
{
|
||||
u8 *addr;
|
||||
@ -1193,7 +1194,7 @@ static void nl80211_del_station_event(struct wpa_driver_nl80211_data *drv,
|
||||
MAC2STR(addr));
|
||||
|
||||
if (is_ap_interface(drv->nlmode) && drv->device_ap_sme) {
|
||||
drv_event_disassoc(drv->ctx, addr);
|
||||
drv_event_disassoc(bss->ctx, addr);
|
||||
return;
|
||||
}
|
||||
|
||||
@ -1202,7 +1203,7 @@ static void nl80211_del_station_event(struct wpa_driver_nl80211_data *drv,
|
||||
|
||||
os_memset(&data, 0, sizeof(data));
|
||||
os_memcpy(data.ibss_peer_lost.peer, addr, ETH_ALEN);
|
||||
wpa_supplicant_event(drv->ctx, EVENT_IBSS_PEER_LOST, &data);
|
||||
wpa_supplicant_event(bss->ctx, EVENT_IBSS_PEER_LOST, &data);
|
||||
}
|
||||
|
||||
|
||||
@ -2155,7 +2156,7 @@ static void do_process_drv_event(struct i802_bss *bss, int cmd,
|
||||
nl80211_new_station_event(drv, bss, tb);
|
||||
break;
|
||||
case NL80211_CMD_DEL_STATION:
|
||||
nl80211_del_station_event(drv, tb);
|
||||
nl80211_del_station_event(drv, bss, tb);
|
||||
break;
|
||||
case NL80211_CMD_SET_REKEY_OFFLOAD:
|
||||
nl80211_rekey_offload_event(drv, tb);
|
||||
|
Loading…
Reference in New Issue
Block a user