iliaa           Tue Aug 16 10:10:15 2005 EDT

  Modified files:              
    /php-src/ext/standard       url.c 
    /php-src/ext/standard/tests/strings url_t.phpt 
    /php-src    NEWS 
  Log:
  Fixed bug #34148 (+,- and . not supported as parts of scheme).
  
  
http://cvs.php.net/diff.php/php-src/ext/standard/url.c?r1=1.87&r2=1.88&ty=u
Index: php-src/ext/standard/url.c
diff -u php-src/ext/standard/url.c:1.87 php-src/ext/standard/url.c:1.88
--- php-src/ext/standard/url.c:1.87     Tue Aug  9 10:17:28 2005
+++ php-src/ext/standard/url.c  Tue Aug 16 10:10:11 2005
@@ -15,7 +15,7 @@
    | Author: Jim Winstead <[EMAIL PROTECTED]>                                  
|
    +----------------------------------------------------------------------+
  */
-/* $Id: url.c,v 1.87 2005/08/09 14:17:28 iliaa Exp $ */
+/* $Id: url.c,v 1.88 2005/08/16 14:10:11 iliaa Exp $ */
 
 #include <stdlib.h>
 #include <string.h>
@@ -107,7 +107,8 @@
                /* 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 {
http://cvs.php.net/diff.php/php-src/ext/standard/tests/strings/url_t.phpt?r1=1.9&r2=1.10&ty=u
Index: php-src/ext/standard/tests/strings/url_t.phpt
diff -u php-src/ext/standard/tests/strings/url_t.phpt:1.9 
php-src/ext/standard/tests/strings/url_t.phpt:1.10
--- php-src/ext/standard/tests/strings/url_t.phpt:1.9   Mon Apr 25 19:47:37 2005
+++ php-src/ext/standard/tests/strings/url_t.phpt       Tue Aug 16 10:10:13 2005
@@ -68,7 +68,8 @@
 'file:/path/to/file',
 'http://1.2.3.4:/abc.asp?a=1&b=2',
 'http://foo.com#bar',
-'scheme:'
+'scheme:',
+'foo+bar://[EMAIL PROTECTED]/bla',
 );
 
     foreach ($sample_urls as $url) {
@@ -662,3 +663,13 @@
   ["scheme"]=>
   string(6) "scheme"
 }
+array(4) {
+  ["scheme"]=>
+  string(7) "foo+bar"
+  ["host"]=>
+  string(4) "bang"
+  ["user"]=>
+  string(3) "baz"
+  ["path"]=>
+  string(4) "/bla"
+}
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.2039&r2=1.2040&ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.2039 php-src/NEWS:1.2040
--- php-src/NEWS:1.2039 Fri Aug 12 11:16:52 2005
+++ php-src/NEWS        Tue Aug 16 10:10:13 2005
@@ -1,6 +1,7 @@
 PHP                                                                        NEWS
 |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
 ?? ??? ????, PHP 6.0
+- Fixed bug #34148 (+,- and . not supported as parts of scheme). (Ilia)
 - cURL improvements (Ilia)
   . Added curl_setopt_array() which allows setting of multiple cURL options.
   . Added CURLINFO_HEADER_OUT to facilitate request retrieval.

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to