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

Jaikiran Pai jpai at openjdk.org
Wed Apr 24 16:14:39 UTC 2024


On Tue, 23 Apr 2024 12:49:15 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 10 additional commits since the last revision:
>> 
>>  - Merge branch 'master' into JDK-7036144
>>  - Back-out Javadoc addition; to be added in a separate issue.
>>  - Document the handling of concatenated streams.
>>  - Merge branch 'master' into JDK-7036144
>>  - 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, we forgot to create a release note for this one (there's still time). Would you be willing to create one, following the instructions here https://openjdk.org/guide/#release-notes? If you need any help, let us know. One of us will review that release note before you can Resolve it to Delivered.

> Hi @jaikiran,
> 
> No problem - please see [JDK-8330995](https://bugs.openjdk.org/browse/JDK-8330995) and let me know if anything else is needed.

Thank you Archie. With inputs from Lance, I've updated the text and the summary of the release note as per the guidelines. You can now mark it as "Resolved", "Delivered".

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

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


More information about the core-libs-dev mailing list