MFH: Fixed bug #34148 (+,- and . not supported as parts of scheme).

diff --git a/ext/standard/tests/strings/url_t.phpt b/ext/standard/tests/strings/url_t.phpt
index 1e4dd3566df84bbd7089761a731df9efde86020a..f6fec50de94443e88f82e67d44cba5121bb6243e 100644 (file)
--- a/ext/standard/tests/strings/url_t.phpt
+++ b/ext/standard/tests/strings/url_t.phpt
@@ -68,7 +68,8 @@ $sample_urls = array (
 'file:/path/to/file',
 'http://1.2.3.4:/abc.asp?a=1&b=2',
 'http://foo.com#bar',
-'scheme:'
+'scheme:',
+'foo+bar://baz@bang/bla',
 );
 
     foreach ($sample_urls as $url) {
@@ -662,3 +663,13 @@ array(1) {
   ["scheme"]=>
   string(6) "scheme"
 }
+array(4) {
+  ["scheme"]=>
+  string(7) "foo+bar"
+  ["host"]=>
+  string(4) "bang"
+  ["user"]=>
+  string(3) "baz"
+  ["path"]=>
+  string(4) "/bla"
+}

diff --git a/ext/standard/url.c b/ext/standard/url.c
index 68280177f70dc65f9b1687b987e1a9231cf3ec10..6e499afe95350a7308998f6c689962316ba0c746 100644 (file)
--- a/ext/standard/url.c
+++ b/ext/standard/url.c
@@ -107,7 +107,8 @@ PHPAPI php_url *php_url_parse_ex(char const *str, int length)
                /* validate scheme */
                p = s;
                while (p < e) {
-                       if (!isalnum(*p)) {
+                       /* scheme = 1*[ lowalpha | digit | "+" | "-" | "." ] */
+                       if (!isalpha(*p) && !isdigit(*p) && *p != '+' && *p != '.' && *p != '-') {
                                if (e + 1 < ue) {
                                        goto parse_port;
                                } else {