httpd: simplified "gzip" test

function                                             old     new   delta
handle_incoming_and_exit                            2830    2807     -23

Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>

diff --git a/networking/httpd.c b/networking/httpd.c
index cad45cd..3fea3f0 100644 (file)
--- a/networking/httpd.c
+++ b/networking/httpd.c
@@ -2081,18 +2081,18 @@ static void handle_incoming_and_exit(const len_and_sockaddr *fromAddr)
 #endif
 #if ENABLE_FEATURE_HTTPD_GZIP
                        if (STRNCASECMP(iobuf, "Accept-Encoding:") == 0) {
-                               char *s = iobuf + sizeof("Accept-Encoding:")-1;
-                               while (*s) {
-                                       ///is "Accept-Encoding: compress,gzip" valid?
-                                       // (that is, no space after ',') -
-                                       // this code won't handle that
-                                       s = skip_whitespace(s);
-                                       if (STRNCASECMP(s, "gzip") == 0)
+                               /* Note: we do not support "gzip;q=0"
+                                * method of _disabling_ gzip
+                                * delivery. No one uses that, though */
+                               const char *s = strstr(iobuf, "gzip");
+                               if (s) {
+                                       // want more thorough checks?
+                                       //if (s[-1] == ' '
+                                       // || s[-1] == ','
+                                       // || s[-1] == ':'
+                                       //) {
                                                content_gzip = 1;
-                                       /* Note: we do not support "gzip;q=0"
-                                        * method of _disabling_ gzip
-                                        * delivery */
-                                       s = skip_non_whitespace(s);
+                                       //}
                                }
                        }
 #endif