#48763 [Com]: ZipArchive produces corrupt OpenOffice.org files

ID: 48763 Comment by: headofsteam at gmx dot com Reported By: dani dot church at gmail dot com Status: Assigned Bug Type: Zip Related Operating System: CentOS 5 PHP Version: 5.2CVS-2009-07-01 (snap) Assigned To: pajoye New Comment: pajoye at php.net thanks for reopening. Don't suppose there is a way to transfer in my comment from #50893 ? Anyway, I have read through this report again and there may be a difference other than the OS and version. This report seems to be highlighting a problem in ZipArchive::AddFromString() I have been using the IMO more useful ZipArchive::AddFile() could that be significant. Previous Comments: ------------------------------------------------------------------------ [2010-02-01 20:19:27] pajoye@php.net reopen and assign to me. ------------------------------------------------------------------------ [2009-11-05 12:13:24] pajoye@php.net Fixed in latest releases too. ------------------------------------------------------------------------ [2009-11-05 12:12:26] pajoye@php.net This bug has been fixed in SVN. Snapshots of the sources are packaged every three hours; this change will be in the next snapshot. You can grab the snapshot at http://snaps.php.net/. Thank you for the report, and for helping us make PHP better. ------------------------------------------------------------------------ [2009-11-05 11:31:50] levelak at post dot cz This bug does not only affect OpenOffice, but also WinRar (I have version 3.80 under Windows). The bug happens whenever a file with more than 255 chars is added via addFromString... eg.: $zip->addFromString("test.txt","asdjdjfdlksjdaf"); //OK $zip->addFromString("test2.txt",str_repeat("A",256); //Corrupt archive The issue is resolved by upgrading to 5.2.11, on 5.2.6 it also works with no problems. ------------------------------------------------------------------------ [2009-07-19 16:37:42] pajoye@php.net Thanks for your patch! I have applied it to all branches and pecl. A pecl release will be done next week. Please note that the patch has been applied upstream as well (libzip repo). I will close the bug once the test is there too. ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at http://bugs.php.net/48763 -- Edit this bug report at http://bugs.php.net/?id=48763&edit=1