|  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #64342 ZipArchive::Close returns false on large file trees
Submitted: 2013-03-03 18:42 UTC Modified: 2013-03-21 20:00 UTC
Avg. Score:5.0 ± 0.0
Reproduced:2 of 2 (100.0%)
Same Version:2 (100.0%)
Same OS:2 (100.0%)
From: kolan_n at mail dot ru Assigned: ab
Status: Closed Package: Zip Related
PHP Version: 5.4.12 OS: Windows
Private report: No CVE-ID:
 [2013-03-03 18:42 UTC] kolan_n at mail dot ru
If you try to archive large file trees using ZipArchive you get false at ZipArchive::close().

ZipArchive::getStatusString says about "unknown error"

Test script:
and dBug (or comment all "new dBug" in files, for example, with Notepad++)
download any archive, containing "large" tree, for example Drupal, and unpack it

$b=new Backuper(
and launch

you will see that it doesn't work

the problem is on the , $this->zip->close() returns false and the files would not be archivated.

Expected result:
$this->zip->close() returns true and the files are archivated

Actual result:
$this->zip->close() returns false and the files are not archivated


Add a Patch

Pull Requests

Add a Pull Request


AllCommentsChangesGit/SVN commitsRelated reports
 [2013-03-12 13:25 UTC] kolan_n at mail dot ru
Unknown error 55703736
 [2013-03-21 18:17 UTC]
It's usually not very handy to debug big scripts. It would rock if you could extract a small snippet representing the wrong behavior.
 [2013-03-21 18:17 UTC]
-Status: Open +Status: Feedback
 [2013-03-21 18:34 UTC] kolan_n at mail dot ru
-Status: Feedback +Status: Open
 [2013-03-21 18:34 UTC] kolan_n at mail dot ru
I replaced this with PHAR and had found a bug in my script - it tryed to archivate a file which is not present beacuse of some errors in hash calculation (used instead of inode in windows).

But the error message should be understandable. It should not be "Unknown error 55703736" it should be "File not found". Also it will be good to add file existence check to ZipArchive::addFile implementation.

Now I am using phar but found some bug in its implementation : if there is .gz in the filename (in the middle of it), the archive type is set to tar.gz regardless of the flags.
 [2013-03-21 18:43 UTC]
-Status: Open +Status: Feedback
 [2013-03-21 18:43 UTC]
Define large files tree. How many entries? No matter the platform, each folder 
can have a files amount limit, or a limit of file handles per process. 

It is pretty easy to create to a small script, create a zip objects, add as many 
files you need to it (from addFromString for example), close it :)

About the error code, mind to show a small code to reproduce it? or is it only a 
file not found on add? addFile uses to check file existences, but only on the 
method call. If the file disappears between add and close, the close method may 
file (compression and co is done on close).
 [2013-03-21 19:29 UTC] kolan_n at mail dot ru
-Status: Feedback +Status: Closed
 [2013-03-21 19:29 UTC] kolan_n at mail dot ru
>addFile uses to check file existences

It only checks wheither dir exists, but there is no checks for file existence

Phar::addFile makes this check, ZipArchive::addFile doesn't.
 [2013-03-21 20:00 UTC]
expand_filepath should do that check, ab, can you check that please? or we can use 
realpath in this case, which does check everything.
 [2013-03-21 20:00 UTC]
-Status: Closed +Status: Assigned -Assigned To: +Assigned To: ab
 [2013-04-10 18:52 UTC]
-Status: Assigned +Status: Closed
 [2013-04-10 18:52 UTC]
Automatic comment on behalf of ab
Log: Fixed bug #64342 ZipArchive::addFile() has to check for file existence
 [2013-04-11 06:47 UTC]
Automatic comment from SVN on behalf of ab
Log: Fixed bug #64342 ZipArchive::addFile() has to check for file existence
PHP Copyright © 2001-2014 The PHP Group
All rights reserved.
Last updated: Wed Apr 23 09:02:23 2014 UTC