Unable to extract zip with extra fields present in local file header

[dasebrook]

A ZIP containing files created with extra fields present in their local file header (such as universal time, and Unix UID/GID) cannot be properly extracted with extractEntryTo or extractAllTo methods.

I believe the fix is to replace the following line in zipFile.js:

entry.setCompressedData(buf.slice(entry.header.offset, entry.header.offset + Utils.Constants.LOCHDR + entry.header.compressedSize + entry.entryName.length));

with:

entry.setCompressedData(buf.slice(entry.header.offset, entry.header.offset + Utils.Constants.LOCHDR + entry.header.compressedSize + entry.entryName.length + buf.readUInt16LE(entry.header.offset + Utils.Constants.LOCEXT)));

[dasebrook]

I believe this might also be a fix for issue 12, as the attached zip file for that issue (dexter.en.zip) contains extra fields in the local header.

[searls]

👍 this fixes the CRC issue for me as well.

[JoshuaGross]

@dasebrook would be helpful for you to submit a pull-request from your branch so folks (like me) could just point their repositories there

[cthackers]

fixed