[jdk8u-dev] Integrated: 8190753: (zipfs): Accessing a large entry (> 2^31 bytes) leads to a negative initial size for ByteArrayOutputStream

Alexey Pavlyutkin duke at openjdk.java.net
Fri Apr 8 07:02:41 UTC 2022


On Fri, 18 Mar 2022 12:24:37 GMT, Alexey Pavlyutkin <duke at openjdk.java.net> wrote:

> Hi!
> 
> Please review the backport of JDK-8190753 to 8u. The patch fixes IllegalArgumentException that takes place on accessing large (>2^31 bytes) entries in zipfs.
> 
> The original patch
> 
> https://github.com/openjdk/jdk11u-dev/pull/545
> 
> was applied with the following changes:
> 
> - changes to absent DeflatingEntryOutputStream class were skipped (in 8u EntryOutputStream handles both RAW and deflating streaming);
> - the tests were updated to not use unsupported APIs like Path.of() and Map.of()
> - timeout value for ZipFSOutputStreamTest was increased because the test is too greedy and may fail by timeout on low performance hosts
> 
> The changes were previously reviewed: https://mail.openjdk.java.net/pipermail/jdk8u-dev/2021-December/014479.html
> 
> Verification/regression: jdk_other (20.04/amd64)

This pull request has now been integrated.

Changeset: 3647d987
Author:    Alexey Pavlyutkin <apavlyutkin at azul.com>
Committer: Yuri Nesterenko <yan at openjdk.org>
URL:       https://git.openjdk.java.net/jdk8u-dev/commit/3647d987abad91eceef72900c71f8f4f55c8e92f
Stats:     365 lines in 3 files changed: 360 ins; 1 del; 4 mod

8190753: (zipfs): Accessing a large entry (> 2^31 bytes) leads to a negative initial size for ByteArrayOutputStream

Reviewed-by: phh, andrew
Backport-of: 8a9cda2d84513ab49a54e1d2a7b530f0bae05c61

-------------

PR: https://git.openjdk.java.net/jdk8u-dev/pull/17


More information about the jdk8u-dev mailing list