From 80f256a568dad7d8f868483bb428c5155ae4585e Mon Sep 17 00:00:00 2001 From: Jouni Malinen Date: Sat, 22 Feb 2014 13:43:08 +0200 Subject: [PATCH] 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 --- src/wps/wps_upnp_web.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/wps/wps_upnp_web.c b/src/wps/wps_upnp_web.c index 11386d882..37bdc8e00 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;