Can you send me a patch against php's 5.3 ext/zip as well as a test case pls (zip+script)?

I can apply it soonish as long as all tests pass.

Would you kindly email me the patch at mnm_raps[at]hotmail[dot]com as i require it urgently for my project. I am developing a website that generates some excel files, zips them and asks the user to download the zip file.

@mnm_raps at hotmail dot com

What prevents you to do that now already? Zip works just fine to create archives, or read them for most of the cases but the one describe in this report.

I don't see a way to attach files to this bug report, so 
I've stored the patch here: http://bramp.net/patches/php5.3-
cde-offset.patch

Now the test case is a little harder. I encounter this bug 
on a 2GB zip file that I didn't not create, and was created 
by a unknown tool. Unfortunately the 2GB zip contains 
sensitive information and I'm not allowed to make it public. 
I could try and make a new test file by hacking up a zip 
file with a hex editor if it would help?

A test script is simply:
<?php
$z = new ZipArchive();
$err = $z->open($filename);
if ($err !== true)
   throw new Exception("Failed to open zip file err $err:" . 
$err);
?>

and $err will be ZIPARCHIVE::ER_NOZIP (Not a zip archive).

Does the tests suite still pass with your patch?

Before my patch 13 tests failed, after my patch the same 13 
tests fail, but my code is now able to read both normal and 
the zip files with the extra records.

The tests that failed do not seem to be related to Zip in any 
way, and I have submitted the failed list to the QA team.

All tests pass here (3 different linux and windows), something is wrong in your settings.

What are the errors?

I'm not too concerned about the 13 tests that fail as they 
are unrelated to this bug report, but on my Debian Testing 
Linux using gcc-4.4.real (Debian 4.4.4-1) and revision 
299964 of the SVN URL: 
http://svn.php.net/repository/php/php-src/tags/php_5_3_2 the 
following failed:

Bug #45877 (Array key '2147483647' left as string) 
[Zend/tests/bug45877.phpt]
Test DateTime::modify() function : usage variation - Passing 
unexpected values to first argument $modify. 
[ext/date/tests/DateTime_modify_variation1.phpt]
Bug #50392 date_create_from_format enforces 6 digits for 'u' 
format character [ext/date/tests/bug50392.phpt]
Test date_modify() function : usage variation - Passing 
unexpected values to second argument $format. 
[ext/date/tests/date_modify_variation2.phpt]
Integer overflow [ext/filter/tests/046.phpt]
Phar: phpinfo display 3 [ext/phar/tests/phpinfo_003.phpt]
realpath_cache_size() and realpath_cache_get() 
[ext/standard/tests/file/realpath_cache.phpt]
proc_open 
[ext/standard/tests/general_functions/proc_open02.phpt]
Test var_dump() function 
[ext/standard/tests/general_functions/var_dump_64bit.phpt]
getmxrr() test [ext/standard/tests/network/getmxrr.phpt]
http-stream test [ext/standard/tests/network/http-
stream.phpt]
Test Blowfish crypt() with invalid rounds 
[ext/standard/tests/strings/crypt_blowfish_invalid_rounds.ph
pt]
htmlentities() / html_entity_decode() #8592 - #9002 table 
test [ext/standard/tests/strings/htmlentities17.phpt]

As I said I submitted these failures, but I'm unsure where 
to find what as submitted as that might contain more 
information.

I mean the Zip tests only, other tests are irrelevant :) So I suppose the zip tests pass with or without your patch, right?

Sorry, perhaps I misunderstood you :)

I run "make test", and I assumed that ran everything 
(including zip tests) and as I did not see the word zip 
printed in the failed tests I assumed they were fine.

Is there a way to run just the zip tests?

But if I did run the zip tests, then yes with and without 
the patch I get the same list of failed tests.

make test TESTS=/home/andrew/php-src/php53/ext/zip/tests should do it.

The issue has been fixed in the meantime. Thanks for the bug
report and the patch.