Closed Bug 781153 Opened 7 years ago Closed 7 years ago
Reader doesn't support zip-in-zip archives
ArchiveReader doesn't support zip-in-zip archives
This patch: 1. implements the ArchiveInputStream as nsISeekableStream. 2. removes the ref between ArchiveInputStream and ArchiveReader. The test_archive_zip_in_zip contains the 42.zip ( http://en.wikipedia.org/wiki/Zip_bomb - http://www.securityfocus.com/bid/3027/exploit/ ). Should I write something about copyright/license? I wrote few lines in the test but maybe it's not enough.
I have replaced the 42.zip with a smaller and easier zip file created by me.
Comment on attachment 651678 [details] [diff] [review] Bug 781153 - ArchiveReader doesn't support zip-in-zip archives Looks good! r=jst
Attachment #651678 - Flags: review?(jst) → review+
Comment on attachment 651678 [details] [diff] [review] Bug 781153 - ArchiveReader doesn't support zip-in-zip archives http://hg.mozilla.org/integration/mozilla-inbound/rev/f450fe554bd6
Attachment #651678 - Flags: checkin? → checkin+
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in before you can comment on or make changes to this bug.