RFR: JDK-8056934: ZipInputStream does not correctly handle local header data descriptors with the optional signature missing
Alan.Bateman at oracle.com
Mon Sep 1 16:05:12 UTC 2014
On 29/08/2014 20:12, Martin Buchholz wrote:
> Hi Xueming and Alan,
> I'd like you to do a code review.
> This seems like an atypical off-by-one, so I'm not sure how it
> happened, and I have the nagging feeling I'm missing something.
> Greg, this java code review contains a python program!
I looked at 22.214.171.124. For ZIP64 then we'll read 24 bytes which is 4 bytes
too much if the signature is not present. The offset of those 4 bytes is
position 20 (ZIP64_EXTHDR - ZIP64_EXTHDR with the current constants).
With ZIP then we'll need 16 bytes and need to unread 4 bytes from
position 12 (EXTHDR - EXTCRC with the current constants).
So I think you're right that the -1, I can only assume that we haven't
encountered zip files that have a data descriptor without the signature.
For the comment in ZipInputStream then it might be better to move that
to readEnd. My preference would be to not include the part starting "As
of 2014-08, phyton ..." as that might not interesting in years to come.
More information about the core-libs-dev