client ext hdr skip if no arg

https://github.com/warmcat/libwebsockets/issues/453

Signed-off-by: Andy Green <andy.green@linaro.org>

diff --git a/changelog b/changelog
index ba12a03..88367db 100644 (file)
--- a/changelog
+++ b/changelog
@@ -1,6 +1,11 @@
 Changelog
 ---------
 
+Fixes
+-----
+
+1) MINOR: d9n't send ext hdr if no exts to discuss
+
 v1.7.3
 ======
 

diff --git a/lib/client.c b/lib/client.c
index 33d990e..1ba809b 100644 (file)
--- a/lib/client.c
+++ b/lib/client.c
@@ -956,7 +956,6 @@ lws_generate_client_handshake(struct lws *wsi, char *pkt)
 
        /* tell the server what extensions we could support */
 
-       p += sprintf(p, "Sec-WebSocket-Extensions: ");
 #ifndef LWS_NO_EXTENSIONS
        ext = context->extensions;
        while (ext && ext->callback) {
@@ -988,13 +987,17 @@ lws_generate_client_handshake(struct lws *wsi, char *pkt)
 
                if (ext_count)
                        *p++ = ',';
+                else
+                        p += sprintf(p, "Sec-WebSocket-Extensions: ");
+
                p += sprintf(p, "%s", ext->client_offer);
                ext_count++;
 
                ext++;
        }
+       if (ext_count)
+               p += sprintf(p, "\x0d\x0a");
 #endif
-       p += sprintf(p, "\x0d\x0a");
 
        if (wsi->ietf_spec_revision)
                p += sprintf(p, "Sec-WebSocket-Version: %d\x0d\x0a",