Integrated: 8278794: Infinite loop in DeflaterOutputStream.finish()

Ravi Reddy rreddy at openjdk.java.net
Fri Mar 18 15:34:42 UTC 2022


On Sun, 13 Mar 2022 14:08:57 GMT, Ravi Reddy <rreddy at openjdk.org> wrote:

> Hi All,
> 
> This review request contains fix for infinite loop issue in DeflaterOutputStream.finish() in an exception scenario.
> 1. The issue is with 'finished' flag not getting set to correct value when there is an IOException in 
>     DeflaterOutputStream.finish() which will result in Infinite loops for next write operations on the same deflater.
> 2. Tighten the condition(to close deflater) in ZipOutputStream using an already existing 'finish' flag in Deflater class.
> 3. Added Inflater exception scenarios also to the test case, renaming test case to CloseInflaterDeflaterTest.java from CloseDeflaterTest.java
> 
> Thanks,
> Ravi

This pull request has now been integrated.

Changeset: ff0b0927
Author:    Ravi Reddy <rreddy at openjdk.org>
Committer: Lance Andersen <lancea at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/ff0b0927a2df8b36f8fd6ed41bd4e20e71a5b653
Stats:     378 lines in 5 files changed: 224 ins; 147 del; 7 mod

8278794: Infinite loop in DeflaterOutputStream.finish()

Reviewed-by: coffeys, lancea

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

PR: https://git.openjdk.java.net/jdk/pull/7804


More information about the core-libs-dev mailing list