DGNum/hostapd
6
0
Fork 0
Code Issues Pull requests 1 Projects Releases Packages Wiki Activity Actions

WPS: Use more defensive design to avoid theoretical NULL deref

Browse source 
Prior to commit 6195adda9b the sm->user
dereference did not exist here. While this is in practice non-NULL,
better use more defensive construction here to avoid NULL pointer
dereference should this ever change.
This commit is contained in:
Jouni Malinen 2010-09-04 21:37:36 +03:00
parent a745b7a775
commit cce1f698e5
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/eap_server/eap_server_wsc.c
View file

@ -128,9 +128,10 @@ static void * eap_wsc_init(struct eap_sm *sm)
wpa_printf(MSG_DEBUG, "EAP-WSC: No AP PIN (password) " wpa_printf(MSG_DEBUG, "EAP-WSC: No AP PIN (password) "
"configured for Enrollee functionality - " "configured for Enrollee functionality - "
"allow for probing capabilities (M1)"); "allow for probing capabilities (M1)");
} else {
cfg.pin = sm->user->password;
cfg.pin_len = sm->user->password_len;
} }
cfg.pin = sm->user->password;
cfg.pin_len = sm->user->password_len;
} }
cfg.assoc_wps_ie = sm->assoc_wps_ie; cfg.assoc_wps_ie = sm->assoc_wps_ie;
cfg.peer_addr = sm->peer_addr; cfg.peer_addr = sm->peer_addr;