> 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
Ravi Reddy has updated the pull request incrementally with one additional commit since the last revision: Modified the format of comments across the test case and also done clean up of test case by using less data providers ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/7804/files - new: https://git.openjdk.java.net/jdk/pull/7804/files/c50aa1bd..ac867083 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7804&range=02 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7804&range=01-02 Stats: 94 lines in 1 file changed: 32 ins; 20 del; 42 mod Patch: https://git.openjdk.java.net/jdk/pull/7804.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7804/head:pull/7804 PR: https://git.openjdk.java.net/jdk/pull/7804