php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Bug #47599 zend_atoi() needs change for 64-bit support
Submitted: 2009-03-08 16:01 UTC Modified: 2009-05-05 16:12 UTC
From: Bjorn dot Wiberg at its dot uu dot se Assigned: fb-req-jani (profile)
Status: Not a bug Package: Scripting Engine problem
PHP Version: 5.2.9 OS: *
Private report: No CVE-ID: None
View Add Comment Developer Edit
 [2009-03-08 16:01 UTC] Bjorn dot Wiberg at its dot uu dot se 
Description:
------------
External sources (http://turin.nss.udel.edu/wiki/dropbox/doku.php?id=documentation:large-files) indicate that zend_atoi() does not handle very large values correctly.

External source supplies proposed fix (change to Zend/zend_operators.c).


Reproduce code:
---------------
Setting post_max_size 10000M or upload_max_filesize 10000M in php.ini.

Expected result:
----------------
Correct handling of very large memory values, e.g. "10000M".

Actual result:
--------------
Limit maxes out at max value of 32 bits instead of specified value.

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2009-05-04 18:50 UTC] jani@php.net 
Also note that this is partially duplicate of bug #27792 (zend_atoi() 
has been replaced with zend_atol() in PHP_5_3)
 [2009-05-05 11:09 UTC] Bjorn dot Wiberg at its dot uu dot se 
As I'm not on a 64-bit system, could someone please try the snapshot (and perhaps propose a test case which uses zend_atoi)?

I'm only reporting it as a third party as it sounded like something that would be good to fix.

Thanks in advance!
 [2009-05-05 16:12 UTC] jani@php.net 
As there is already report about LFS issue and I know the ini issue is 
solved I'm closing this. Please don't report bugs you can not reproduce 
yourself.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Sat May 18 09:01:33 2024 UTC