The default bug view has changed. See this FAQ.

ArchiveReader doesn't support zip-in-zip archives

RESOLVED FIXED in mozilla17

Status

()

Core
DOM
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: baku, Assigned: baku)

Tracking

unspecified
mozilla17
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

5 years ago
ArchiveReader doesn't support zip-in-zip archives
(Assignee)

Comment 1

5 years ago
Created attachment 650212 [details] [diff] [review]
Bug 781153 - 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.
Attachment #650212 - Flags: review?(jst)
(Assignee)

Comment 2

5 years ago
Created attachment 651678 [details] [diff] [review]
Bug 781153 - ArchiveReader doesn't support zip-in-zip archives

I have replaced the 42.zip with a smaller and easier zip file created by me.
Attachment #650212 - Attachment is obsolete: true
Attachment #650212 - Flags: review?(jst)
Attachment #651678 - Flags: review?(jst)
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+
(Assignee)

Updated

5 years ago
Attachment #651678 - Flags: checkin?
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+
Assignee: nobody → amarchesini
https://hg.mozilla.org/mozilla-central/rev/f450fe554bd6
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
Depends on: 783863
You need to log in before you can comment on or make changes to this bug.