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

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

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 > >

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