RFR: 8272746: ZipFile can't open big file (NegativeArraySizeException) [v3]
Lance Andersen
lancea at openjdk.java.net
Fri Jan 14 23:59:39 UTC 2022
On Fri, 14 Jan 2022 11:05:06 GMT, Masanori Yano <myano at openjdk.org> wrote:
>> Could you please review the JDK-8272746 bug fixes?
>> Since the array index is of type int, the overflow occurs when the value of end.cenlen is too large because of too many entries.
>> It is necessary to read a part of the CEN from the file to fix the problem fundamentally, but the way will require an extensive fix and degrade performance.
>> In practical terms, the size of the central directory rarely grows that large. So, I fixed it to check the size of the central directory and throw an exception if it is too large.
>
> Masanori Yano has updated the pull request incrementally with one additional commit since the last revision:
>
> 8272746: ZipFile can't open big file (NegativeArraySizeException)
Thank you for making the changes. Overall it looks much better
Please add comments describing your constants as well as a comment describing the intent of your setup and test methods
-------------
PR: https://git.openjdk.java.net/jdk/pull/6927
More information about the core-libs-dev
mailing list