RFR: 8255380: (zipfs) ZipFileSystem::readExtra can fail if zipinfo-time is not set to false [v2]
Lance Andersen
lancea at openjdk.java.net
Sun Nov 1 22:59:09 UTC 2020
On Sun, 1 Nov 2020 22:21:25 GMT, Claes Redestad <redestad at openjdk.org> wrote:
> LGTM. Some nits inline.
>
> I guess the test can be cause for issues in some test systems since it needs to write out the 4Gb+ file. Is this why you've only enabled it on linux and mac? Perhaps someone might have ideas on how to improve this.
To validate the test, requires info-zip which comes on Mac and linux. It is not included with windows. There are no issues if the Zip is created via java.util.zip or Zip FS
> src/jdk.zipfs/share/classes/jdk/nio/zipfs/ZipFileSystem.java line 2993:
>
>> 2991: // We need to read the LOC extra data and the LOC offset was obtained
>> 2992: // from the EXTID_ZIP64 field.
>> 2993: if(hasZip64LocOffset) {
>
> Suggestion:
>
> if (hasZip64LocOffset) {
Resolved
-------------
PR: https://git.openjdk.java.net/jdk/pull/987
More information about the core-libs-dev
mailing list