php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Doc Bug #17707 ini_set and upload_max_filesize
Submitted: 2002-06-11 12:31 UTC Modified: 2002-07-09 02:35 UTC
From: tr909 at bigfoot dot com Assigned:
Status: Closed Package: Documentation problem
PHP Version: 4.1.2 OS: linux 7.1
Private report: No CVE-ID: None
 [2002-06-11 12:31 UTC] tr909 at bigfoot dot com
Hello great PHP,

i stumble over and over on 'upload_max_filesize' 
>  ini_set('upload_max_filesize','8M');
>  var_dump( ini_get('upload_max_filesize') );

this returns a nice string(2) "8M", but if you do
a var_dump($HTTP_POST_FILES) after uploading
a 3MB movie it returns:

> array(1) { ["ufile"]=>
>   array(4) { ["name"]=> string(19) "PN.mpg"
>     ["type"]=> string(10) "video/mpeg"
>     ["tmp_name"]=> string(4) "none"
>     ["size"]=> int(0) 
>   }
> }

ONLY if you change upload_max_filesize in php.ini then
will it listen to the correct size....

ini_set() function documention says... 
'Returns the old value on success, FALSE on failure.'

this means that ini_set() should return FALSE if you
cannot set upload_max_filesize in your script OR that
the max-upload-size should be set to the new value and
accepted. at the moment this is very confusing because 
it returns the old value. Even init_get() returns the newly 
set value. In the end upload_m-fsize is as set in the ini 
file.

 Tr909

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2002-06-11 13:35 UTC] sander@php.net
The settings will be changed if you're using ini_set() but as the uploaded file will be accepted BEFORE your script runs, this stuff has no effect. Same applies for post_max_size and a couple of other settings.
 [2002-06-12 09:58 UTC] tr909 at bigfoot dot com
Will check with .htaccess (thanx 4 the swift reply)
 [2002-06-12 14:12 UTC] philip@php.net
On a related note, the ini_set() configuration options table is autogenerated from the PHP source.  Sometimes they are generated "wrongly" as the PHP source (main.c) lists questionable values.  For example, magic_quotes_gpc shows PHP_INI_ALL when in the real-world this isn't the case.
 [2002-06-12 16:46 UTC] goba@php.net
So then the sources should be corrected. Goba
 [2002-07-09 02:35 UTC] philip@php.net
This was fixed in main.c by sniper, see 1.442 here:
  http://bonsai.php.net/cvslog.cgi?file=/php4/main/main.c

Changed: PHP_INI_ALL -> PHP_INI_SYSTEM

Changing this from bogus to closed.  The ini_set() docs are being worked on and more main.c changes may be on the way ;)
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Mon May 20 06:01:34 2024 UTC