RFR: 7036144: GZIPInputStream readTrailer uses faulty available() test for end-of-stream [v6]

[Archie Cobbs]

On Wed, 6 Mar 2024 14:14:56 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

>> Archie Cobbs has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains six additional commits since the last revision:
>> 
>>  - Merge branch 'master' into JDK-7036144
>>  - Merge branch 'master' into JDK-7036144
>>  - Address third round of review comments.
>>  - Address second round of review comments.
>>  - Address review comments.
>>  - Fix bug in GZIPInputStream when underlying available() returns short.
>
> Hello Archie, I hope to finish off running the necessary analysis to see if there is any obvious impact because of this change, in the coming days. Based on the intial runs, the changes proposed in this PR look OK to me.
> 
> In the meantime, given the nature of this change, I am marking this as requiring a CSR. Would you be willing to come up with the CSR text for this (https://wiki.openjdk.org/display/csr/Main)?

Hi @jaikiran,

> Would you be willing to come up with the CSR text for this

Happy to - please see [JDK-8327489](https://bugs.openjdk.org/browse/JDK-8327489).

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

PR Comment: https://git.openjdk.org/jdk/pull/17113#issuecomment-1981533001