clones/php-src
1
0
Fork 0
mirror of https://github.com/php/php-src.git synced 2024-09-21 18:07:23 +00:00
Code Issues Packages Projects Releases Wiki Activity

- Fixed bug #53180 (post_max_size=0 not disabling the limit when the content

Browse Source 
type is application/x-www-form-urlencoded or is not registered with PHP).
This commit is contained in:
Gustavo André dos Santos Lopes 2010-10-27 14:56:51 +00:00
parent be77a48a71
commit da400e7500
2 changed files with 21 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
main/SAPI.c
View File

@ -194,7 +194,7 @@ SAPI_API SAPI_POST_READER_FUNC(sapi_read_standard_form_data)
int read_bytes;
int allocated_bytes=SAPI_POST_BLOCK_SIZE+1;
if (SG(request_info).content_length > SG(post_max_size)) {
if ((SG(post_max_size) > 0) && (SG(request_info).content_length > SG(post_max_size))) {
php_error_docref(NULL TSRMLS_CC, E_WARNING, "POST Content-Length of %ld bytes exceeds the limit of %ld bytes",
SG(request_info).content_length, SG(post_max_size));
return;
@ -207,7 +207,7 @@ SAPI_API SAPI_POST_READER_FUNC(sapi_read_standard_form_data)
break;
}
SG(read_post_bytes) += read_bytes;
if (SG(read_post_bytes) > SG(post_max_size)) {
if ((SG(post_max_size) > 0) && (SG(read_post_bytes) > SG(post_max_size))) {
php_error_docref(NULL TSRMLS_CC, E_WARNING, "Actual POST length does not match Content-Length, and exceeds %ld bytes", SG(post_max_size));
break;
}

19
tests/basic/bug53180.phpt Normal file
View File

@ -0,0 +1,19 @@
--TEST--
Bug #53180 (post_max_size=0 partly not working)
--INI--
post_max_size=0
--POST--
email=foo&password=bar&submit=Log+on
--FILE--
<?php
var_dump($_POST);
?>
--EXPECT--
array(3) {
["email"]=>
string(3) "foo"
["password"]=>
string(3) "bar"
["submit"]=>
string(6) "Log on"
}