From: William A. Rowe Jr <wrowe@apache.org>
Date: Sun, 21 Aug 2016 16:12:49 +0000 (+0000)
Subject: Yann is correct, % is distinct from reserved and unreserved
X-Git-Tag: 2.5.0-alpha~1221
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=3a1ebf13aca557baf95d5a125bc74a8bee356b94;p=apache

Yann is correct, % is distinct from reserved and unreserved

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1757062 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/server/gen_test_char.c b/server/gen_test_char.c
index 5003dc2962..ae7e18a7c6 100644
--- a/server/gen_test_char.c
+++ b/server/gen_test_char.c
@@ -143,7 +143,8 @@ int main(int argc, char *argv[])
          * and unreserved (2.3) that are possible somewhere within a URI.
          * Spec requires all others to be %XX encoded, including obs-text.
          */
-        if (c && (strchr(":/?#[]@"                        /* gen-delims */ 
+        if (c && (strchr("%"                              /* pct-encode */
+                         ":/?#[]@"                        /* gen-delims */ 
                          "!$&'()*+,;="                    /* sub-delims */
                          "-._~", c) || apr_isalnum(c))) { /* unreserved */
             flags |= T_URI_RFC3986;