mirror of
https://github.com/php/php-src.git
synced 2025-08-18 15:08:55 +02:00
# Revert bug fix (will commit it again on 5.3 branch when 5.3.6 will be released)
This commit is contained in:
parent
ee832705d1
commit
e68e2893c0
3 changed files with 0 additions and 40 deletions
4
NEWS
4
NEWS
|
@ -1,10 +1,6 @@
|
|||
PHP NEWS
|
||||
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||
10 Mar 2011, PHP 5.3.6RC3
|
||||
- Core:
|
||||
. Fixed bug #54180 (parse_url() incorrectly parses path when ? in fragment).
|
||||
(tomas dot brastavicius at quantum dot lt, Pierrick)
|
||||
|
||||
- Shmop extension:
|
||||
. Fixed bug #54193 (Integer overflow in shmop_read()). (Felipe)
|
||||
Reported by Jose Carlos Norte <jose at eyeos dot org> (CVE-2011-1092)
|
||||
|
|
|
@ -1,32 +0,0 @@
|
|||
--TEST--
|
||||
Bug #54180 (parse_url() incorrectly parses path when ? in fragment)
|
||||
--FILE--
|
||||
<?php
|
||||
|
||||
var_dump(parse_url("http://example.com/path/script.html?t=1#fragment?data"));
|
||||
var_dump(parse_url("http://example.com/path/script.html#fragment?data"));
|
||||
|
||||
?>
|
||||
--EXPECTF--
|
||||
array(5) {
|
||||
["scheme"]=>
|
||||
string(4) "http"
|
||||
["host"]=>
|
||||
string(11) "example.com"
|
||||
["path"]=>
|
||||
string(17) "/path/script.html"
|
||||
["query"]=>
|
||||
string(3) "t=1"
|
||||
["fragment"]=>
|
||||
string(13) "fragment?data"
|
||||
}
|
||||
array(4) {
|
||||
["scheme"]=>
|
||||
string(4) "http"
|
||||
["host"]=>
|
||||
string(11) "example.com"
|
||||
["path"]=>
|
||||
string(17) "/path/script.html"
|
||||
["fragment"]=>
|
||||
string(13) "fragment?data"
|
||||
}
|
|
@ -316,10 +316,6 @@ PHPAPI php_url *php_url_parse_ex(char const *str, int length)
|
|||
pp = strchr(s, '#');
|
||||
|
||||
if (pp && pp < p) {
|
||||
if (pp - s) {
|
||||
ret->path = estrndup(s, (pp-s));
|
||||
php_replace_controlchars_ex(ret->path, (pp - s));
|
||||
}
|
||||
p = pp;
|
||||
goto label_parse;
|
||||
}
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue