Req #55857 [Ana]: ftp_size on large files

From: Date: Mon, 23 Jul 2018 15:41:30 +0000
Subject: Req #55857 [Ana]: ftp_size on large files
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-216416@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=55857&edit=1 ID: 55857 Updated by: cmb@php.net Reported by: websupport at osite dot de Summary: ftp_size on large files Status: Analyzed Type: Feature/Change Request Package: FTP related -Operating System: linux +Operating System: Windows 64bit -PHP Version: 5.3.8 +PHP Version: 7.3.0alpha4 Block user comment: N Private report: N New Comment: This is fixed for 64bit POSIX systems as of PHP 5.4.20[1], and is unlikely to get fixed for 32bit systems. However, it won't work for 64bit Windows, since the function uses atol()[2] which returns a long. [1] <http://git.php.net/?p=php-src.git;a=commit;h=a93a462> [2] <https://github.com/php/php-src/blob/php-7.2.8/ext/ftp/ftp.c#L1144> Previous Comments: ------------------------------------------------------------------------ [2013-07-23 17:36:32] lang at b1-systems dot de The suggested patch uses long instead. Double is not a good value for file sizes as discussed here: https://bugs.php.net/bug.php?id=44522 https://github.com/php/php-src/pull/372 ------------------------------------------------------------------------ [2013-07-23 16:48:58] daltoncaughell at gmail dot com Stumbled across this issue just thought I would toss in my solution. Just queries the ftp server for the size and converts the resulting string to a double rather than an int. $size = 0; $result = ftp_raw($conn_id, "SIZE " . <file path>); $parts = explode(" ", $result[0]); if($parts[0] === "213") { $size = doubleval($parts[1]); } ------------------------------------------------------------------------ [2011-10-06 08:58:23] pajoye@php.net It depends on the underlying API used for the network operations, the php's stream implementation. PHP's stream does not support yet LFS and cannot support it easily by only using long instead of int (while this breaks the api signature too). Suspended until we have lfs support. ------------------------------------------------------------------------ [2011-10-06 07:54:34] websupport at osite dot de Description: ------------ ftp_size does not work on large files > 2gb event not on 64bit systems can be fixed with the following patch ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=55857&edit=1

« previous php.bugs (#216416) next »