aboutsummaryrefslogtreecommitdiffstats
path: root/src/wps
diff options
context:
space:
mode:
authorJouni Malinen <j@w1.fi>2014-02-22 11:43:08 (GMT)
committerJouni Malinen <j@w1.fi>2014-02-22 12:07:23 (GMT)
commit80f256a568dad7d8f868483bb428c5155ae4585e (patch)
treeb69c223afd8404d3324bd7917d7accc0b991f124 /src/wps
parent29513e461a7114ac31c713caf846fded585459f7 (diff)
downloadhostap-80f256a568dad7d8f868483bb428c5155ae4585e.zip
hostap-80f256a568dad7d8f868483bb428c5155ae4585e.tar.gz
hostap-80f256a568dad7d8f868483bb428c5155ae4585e.tar.bz2
WPS: Remove unnecessary filename NULL check
The caller of the GET parser is checking this already and the GET case was the only one that ended up doing the duplicated validation step. Signed-off-by: Jouni Malinen <j@w1.fi>
Diffstat (limited to 'src/wps')
-rw-r--r--src/wps/wps_upnp_web.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/src/wps/wps_upnp_web.c b/src/wps/wps_upnp_web.c
index 11386d8..37bdc8e 100644
--- a/src/wps/wps_upnp_web.c
+++ b/src/wps/wps_upnp_web.c
@@ -324,8 +324,6 @@ static void web_connection_parse_get(struct upnp_wps_device_sm *sm,
* It is not required that filenames be case insensitive but it is
* allowed and cannot hurt here.
*/
- if (filename == NULL)
- filename = "(null)"; /* just in case */
if (os_strcasecmp(filename, UPNP_WPS_DEVICE_XML_FILE) == 0) {
wpa_printf(MSG_DEBUG, "WPS UPnP: HTTP GET for device XML");
req = GET_DEVICE_XML_FILE;