On Wed, 27 Mar 2024 17:43:41 GMT, Liam Miller-Cushon <[email protected]> wrote:
>> This change fixes a zip64 bug in the launcher that is prevent it from
>> reading the manifest of jars where the 'relative offset of local header'
>> field in the central directory entry is >4GB. As described in APPNOTE.TXT
>> 4.5.3, the offset is too large to be stored in the central directory it is
>> stored in a 'Zip64 Extended Information Extra Field'.
>
> Liam Miller-Cushon has updated the pull request incrementally with one
> additional commit since the last revision:
>
> Make cendsk an unsigned short
src/java.base/share/native/libjli/parse_manifest.c line 505:
> 503: Byte *base = p + CENHDR + CENNAM(p);
> 504: jlong offset = 0;
> 505: while (offset < cenext) {
Any reason why a loop is need here?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18479#discussion_r1544726976
