From 56906d06c79a9ce0f09dc1d2732edd6f68ea2691 Mon Sep 17 00:00:00 2001 From: Jouni Malinen Date: Sat, 15 Aug 2015 22:24:15 +0300 Subject: [PATCH] WPS ER: Fix SSDP CACHE-CONTROL line parser Incorrect number of bytes was skipped from the beginning of the line which resulted in the loop skipping spaces doing nothing. However, the following operation was simply looking for the max-age parameter with os_strstr(), so this did not have any effect on functionality. Fix the number of bytes to skip and remove the unneeded loop to skip spaces. Signed-off-by: Jouni Malinen --- src/wps/wps_er_ssdp.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/wps/wps_er_ssdp.c b/src/wps/wps_er_ssdp.c index e381fecbd..280b2b3bd 100644 --- a/src/wps/wps_er_ssdp.c +++ b/src/wps/wps_er_ssdp.c @@ -78,9 +78,7 @@ static void wps_er_ssdp_rx(int sd, void *eloop_ctx, void *sock_ctx) if (os_strstr(start, "ssdp:byebye")) byebye = 1; } else if (os_strncasecmp(start, "CACHE-CONTROL:", 14) == 0) { - start += 9; - while (*start == ' ') - start++; + start += 14; pos2 = os_strstr(start, "max-age="); if (pos2 == NULL) continue;