jeltz/hostap
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Remove unnecessary NULL check to make function more consistent

Browse source 
Static analyzers may warn about dereference before NULL check in
wpas_network_disabled() due to the new code added to check
wpa_s->p2p_mgmt. wpa_s cannot be NULL here, so remove the unneeded check
for it later in the function. (CID 106124)

Signed-off-by: Jouni Malinen <j@w1.fi>
This commit is contained in:
Jouni Malinen 2015-03-01 22:15:44 +02:00
parent bfc048b48f
commit 9feadba141
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

@ -4986,7 +4986,7 @@ int wpas_network_disabled(struct wpa_supplicant *wpa_s, struct wpa_ssid *ssid)
if (ssid->disabled) if (ssid->disabled)
return 1; return 1;
if (wpa_s && wpa_s->drv_capa_known) if (wpa_s->drv_capa_known)
drv_enc = wpa_s->drv_enc; drv_enc = wpa_s->drv_enc;
else else
drv_enc = (unsigned int) -1; drv_enc = (unsigned int) -1;