On Wed, 27 Mar 2024 03:18:42 GMT, Jiangli Zhou <jian...@openjdk.org> wrote:
>> Liam Miller-Cushon has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Fix disk number size > > src/java.base/share/native/libjli/manifest_info.h line 59: > >> 57: #define ZIP64_EXTID 1 // Extra field Zip64 header ID >> 58: >> 59: #define ZIP64_EXTMAXLEN 36 // Maximum Zip64 extra field length > > The fields described in APPNOTE-6.3.9.TXT 4.5.3 are total 32 bytes. Any other > additional fields in the Zip64 extended information? > > > Value Size Description > ----- ---- ----------- > (ZIP64) 0x0001 2 bytes Tag for this "extra" block type > Size 2 bytes Size of this "extra" block > Original > Size 8 bytes Original uncompressed file size > Compressed > Size 8 bytes Size of compressed data > Relative Header > Offset 8 bytes Offset of local header record > Disk Start > Number 4 bytes Number of the disk on which > this file starts Thanks for the catch, I had missed that the disk start number is 4 bytes and not 8. I pushed a commit. I also removed some unused references to the disk number, which is only being used to validate the size of the zip64 extended info. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/18479#discussion_r1541404663