codec/url.c: Always encode whitespace characters.

Vertical whitespace is obviously bad, so this is a longstanding bug; but
all whitespace should really be escaped.

diff --git a/codec/url.c b/codec/url.c
index f502f7c..4e7f190 100644 (file)
--- a/codec/url.c
+++ b/codec/url.c
@@ -67,7 +67,8 @@ static void encode(url_ectx *ctx, dstr *d, const char *p)
        DPUTC(d, '+');
        break;
       default:
-       if ((ctx->f & URLF_LAX) || isalnum((unsigned char)*p))
+       if (!isspace((unsigned char)*p) &&
+           ((ctx->f & URLF_LAX) || isalnum((unsigned char)*p)))
          goto safe;
        else
          goto unsafe;