Re: RFR: 8278794: Infinite loop in DeflaterOutputStream.finish() [v3]

2022-03-17 Thread Lance Andersen
On Thu, 17 Mar 2022 16:56:15 GMT, Ravi Reddy 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 i

Re: RFR: 8278794: Infinite loop in DeflaterOutputStream.finish() [v3]

2022-03-17 Thread Ravi Reddy
> 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

Re: RFR: 8278794: Infinite loop in DeflaterOutputStream.finish() [v2]

2022-03-17 Thread Ravi Reddy
On Wed, 16 Mar 2022 16:34:37 GMT, Ravi Reddy 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 i

Re: RFR: 8278794: Infinite loop in DeflaterOutputStream.finish() [v2]

2022-03-16 Thread Lance Andersen
On Wed, 16 Mar 2022 16:34:37 GMT, Ravi Reddy 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 i

Re: RFR: 8278794: Infinite loop in DeflaterOutputStream.finish() [v2]

2022-03-16 Thread Ravi Reddy
On Wed, 16 Mar 2022 11:59:44 GMT, Sean Coffey wrote: >> Ravi Reddy has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Modified write(byte []b) to write(byte[] b) to maintain same signature >> across the test case > > test/jdk/java/util/zip

Re: RFR: 8278794: Infinite loop in DeflaterOutputStream.finish() [v2]

2022-03-16 Thread Ravi Reddy
> 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

Re: RFR: 8278794: Infinite loop in DeflaterOutputStream.finish() [v2]

2022-03-16 Thread Sean Coffey
On Wed, 16 Mar 2022 16:31:36 GMT, Ravi Reddy 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 i

Re: RFR: 8278794: Infinite loop in DeflaterOutputStream.finish()

2022-03-16 Thread Sean Coffey
On Sun, 13 Mar 2022 14:08:57 GMT, Ravi Reddy 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 >

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

2022-03-13 Thread Ravi Reddy
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