Re: [PATCH] archive-zip: Add zip64 headers when file size is too large for 32 bits

From: Johannes Sixt <j6t@kdbg.org>
Date: Apr 23, 2017, 07:27 UTC
Message-ID: <daa66f7e-b77e-3a27-a6f1-7d9059ab71b7@kdbg.org>
In-Reply-To: <alpine.DEB.2.11.1704230737380.29888@perkele.intern.softwolves.pp.se>

Am 23.04.2017 um 08:42 schrieb Peter Krefting:

> René Scharfe:
>> The offset is only needed in the ZIP64 extra record for the central
>> header (in zip_dir) -- the local header has no offset field.

Good point.

> The zip64 local header does have an offset field, though. I thought that
> was the zip_offset value, but that doesn't make sense, I'm not quite
> sure what it is supposed to store. I need to investigate that further, I
> assume.

Let's get the naming straight: There is no "zip64 local header". There is a "zip64 extra record" for the "zip local header". The zip64 extra data record has an offset field, but since the local header does not have an offset field, the offset field in the corresponding zip64 extra data record is always omitted.

-- Hannes