iliaa Thu Sep 28 14:52:31 2006 UTC
Modified files: (Branch: PHP_5_2)
/php-src/ext/standard url.c
/php-src/ext/standard/tests/strings url_t.phpt
/php-src NEWS
Log:
Fixed bug #38859 (parse_url() fails if passing '@' in passwd).
http://cvs.php.net/viewvc.cgi/php-src/ext/standard/url.c?r1=1.86.2.5.2.5&r2=1.86.2.5.2.6&diff_format=u
Index: php-src/ext/standard/url.c
diff -u php-src/ext/standard/url.c:1.86.2.5.2.5
php-src/ext/standard/url.c:1.86.2.5.2.6
--- php-src/ext/standard/url.c:1.86.2.5.2.5 Thu Sep 28 12:55:15 2006
+++ php-src/ext/standard/url.c Thu Sep 28 14:52:30 2006
@@ -15,7 +15,7 @@
| Author: Jim Winstead <[EMAIL PROTECTED]>
|
+----------------------------------------------------------------------+
*/
-/* $Id: url.c,v 1.86.2.5.2.5 2006/09/28 12:55:15 tony2001 Exp $ */
+/* $Id: url.c,v 1.86.2.5.2.6 2006/09/28 14:52:30 iliaa Exp $ */
#include <stdlib.h>
#include <string.h>
@@ -211,7 +211,7 @@
}
/* check for login and password */
- if ((p = memchr(s, '@', (e-s)))) {
+ if ((p = zend_memrchr(s, '@', (e-s)))) {
if ((pp = memchr(s, ':', (p-s)))) {
if ((pp-s) > 0) {
ret->user = estrndup(s, (pp-s));
http://cvs.php.net/viewvc.cgi/php-src/ext/standard/tests/strings/url_t.phpt?r1=1.9.2.3&r2=1.9.2.3.2.1&diff_format=u
Index: php-src/ext/standard/tests/strings/url_t.phpt
diff -u php-src/ext/standard/tests/strings/url_t.phpt:1.9.2.3
php-src/ext/standard/tests/strings/url_t.phpt:1.9.2.3.2.1
--- php-src/ext/standard/tests/strings/url_t.phpt:1.9.2.3 Sun Feb 12
16:39:44 2006
+++ php-src/ext/standard/tests/strings/url_t.phpt Thu Sep 28 14:52:30 2006
@@ -71,6 +71,7 @@
'scheme:',
'foo+bar://[EMAIL PROTECTED]/bla',
'gg:9130731',
+'http://user:@[EMAIL PROTECTED]/path?argument?value#etc',
);
foreach ($sample_urls as $url) {
@@ -525,11 +526,11 @@
["scheme"]=>
string(4) "http"
["host"]=>
- string(19) "[EMAIL PROTECTED]"
+ string(11) "www.php.net"
["port"]=>
int(80)
["user"]=>
- string(6) "secret"
+ string(14) "[EMAIL PROTECTED]"
["path"]=>
string(10) "/index.php"
["query"]=>
@@ -685,6 +686,22 @@
["path"]=>
string(7) "9130731"
}
+array(7) {
+ ["scheme"]=>
+ string(4) "http"
+ ["host"]=>
+ string(4) "host"
+ ["user"]=>
+ string(4) "user"
+ ["pass"]=>
+ string(5) "@pass"
+ ["path"]=>
+ string(5) "/path"
+ ["query"]=>
+ string(14) "argument?value"
+ ["fragment"]=>
+ string(3) "etc"
+}
string(4) "http"
string(11) "www.php.net"
int(80)
http://cvs.php.net/viewvc.cgi/php-src/NEWS?r1=1.2027.2.547.2.271&r2=1.2027.2.547.2.272&diff_format=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.2027.2.547.2.271 php-src/NEWS:1.2027.2.547.2.272
--- php-src/NEWS:1.2027.2.547.2.271 Thu Sep 28 12:57:56 2006
+++ php-src/NEWS Thu Sep 28 14:52:31 2006
@@ -16,6 +16,7 @@
the imap library). (Ilia)
- Fixed bug #38904 (apache2filter changes cwd to /). (Ilia, bjori)
- Fixed bug #38891 (get_headers() do not work with curl-wrappers). (Ilia)
+- Fixed bug #38859 (parse_url() fails if passing '@' in passwd). (Tony)
- Fixed bug #38844 (curl_easy_strerror() is defined only since cURL 7.12.0).
(Tony)
- Fixed bug #38808 ("maybe ref" issue for current() and others). (Dmitry)
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
