Amazing-Mirror/fragattacks
1
0
Fork 0
mirror of https://github.com/vanhoefm/fragattacks.git synced 2025-02-17 17:43:06 -05:00
Code Issues Projects Releases Wiki Activity

FT: Fix potential NULL pointer dereference in MDE addition

Browse Source 
The bss variable in this function might be NULL, so make the FT MDE
addition case conditional on a BSS entry being available.

Fixes: 3dc3afe298f0 ("FT: Add MDE to assoc request IEs in connect params")
Signed-off-by: Jouni Malinen <j@w1.fi>
This commit is contained in:
Jouni Malinen 2018-06-05 14:40:18 +03:00
parent 661afb2edd
commit 88bf44be42
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
wpa_supplicant/wpa_supplicant.c
View File

@ -2748,7 +2748,7 @@ static u8 * wpas_populate_assoc_ies(
* Add MDIE under these conditions: the network profile allows FT, * Add MDIE under these conditions: the network profile allows FT,
* the AP supports FT, and the mobility domain ID matches. * the AP supports FT, and the mobility domain ID matches.
*/ */
if (wpa_key_mgmt_ft(wpa_sm_get_key_mgmt(wpa_s->wpa))) { if (bss && wpa_key_mgmt_ft(wpa_sm_get_key_mgmt(wpa_s->wpa))) {
const u8 *mdie = wpa_bss_get_ie(bss, WLAN_EID_MOBILITY_DOMAIN); const u8 *mdie = wpa_bss_get_ie(bss, WLAN_EID_MOBILITY_DOMAIN);
if (mdie && mdie[1] >= MOBILITY_DOMAIN_ID_LEN) { if (mdie && mdie[1] >= MOBILITY_DOMAIN_ID_LEN) {