iliaa           Tue Aug 16 10:12:15 2005 EDT

  Modified files:              (Branch: PHP_4_4)
    /php-src/ext/standard       url.c 
    /php-src/ext/standard/tests/strings url_t.phpt 
    /php-src    NEWS 
  Log:
  MFH: Fixed bug #34148 (+,- and . not supported as parts of scheme).
  
  
http://cvs.php.net/diff.php/php-src/ext/standard/url.c?r1=1.58.2.21&r2=1.58.2.21.2.1&ty=u
Index: php-src/ext/standard/url.c
diff -u php-src/ext/standard/url.c:1.58.2.21 
php-src/ext/standard/url.c:1.58.2.21.2.1
--- php-src/ext/standard/url.c:1.58.2.21        Wed May 25 23:56:45 2005
+++ php-src/ext/standard/url.c  Tue Aug 16 10:12:13 2005
@@ -15,7 +15,7 @@
    | Author: Jim Winstead <[EMAIL PROTECTED]>                                  
|
    +----------------------------------------------------------------------+
  */
-/* $Id: url.c,v 1.58.2.21 2005/05/26 03:56:45 iliaa Exp $ */
+/* $Id: url.c,v 1.58.2.21.2.1 2005/08/16 14:12:13 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.3.2.7&r2=1.3.2.7.2.1&ty=u
Index: php-src/ext/standard/tests/strings/url_t.phpt
diff -u php-src/ext/standard/tests/strings/url_t.phpt:1.3.2.7 
php-src/ext/standard/tests/strings/url_t.phpt:1.3.2.7.2.1
--- php-src/ext/standard/tests/strings/url_t.phpt:1.3.2.7       Mon Apr 25 
19:49:27 2005
+++ php-src/ext/standard/tests/strings/url_t.phpt       Tue Aug 16 10:12:13 2005
@@ -70,7 +70,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) {
@@ -664,3 +665,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.1247.2.920.2.25&r2=1.1247.2.920.2.26&ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.1247.2.920.2.25 php-src/NEWS:1.1247.2.920.2.26
--- php-src/NEWS:1.1247.2.920.2.25      Mon Aug 15 06:47:17 2005
+++ php-src/NEWS        Tue Aug 16 10:12:13 2005
@@ -1,6 +1,7 @@
 PHP 4                                                                      NEWS
 |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
 ?? ??? 2005, Version 4.4.1
+- Fixed bug #34148 (+,- and . not supported as parts of scheme). (Ilia)
 - Fixed bug #34068 (Numeric string as array key not cast to integer in 
   wddx_deserialize()). (Ilia)
 - Fixed bug #34064 (arr[] as param to function is allowed only if function 

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

Reply via email to