[jdk21u-dev] RFR: 8343855: HTTP/2 ConnectionWindowUpdateSender may miss some unprocessed DataFrames from closed streams [v5]

[Paul Hohensee]

On Wed, 16 Apr 2025 15:01:10 GMT, Paul Hohensee <phh at openjdk.org> wrote:

>> Backport for parity with Oracle 21.0.8. Passes tier2 and modified tests. Low risk: the patch /1 locks existing unlocked code, which is at worst a performance issue, but necessary for correctness, /2 checks for a closed stream before pushing a data frame into it, see pushDataFrame() in Stream.java.
>> 
>> Clean except for whitespace change in Stream.java from 8309118, 8309118 isn't in 21u.
>
> Paul Hohensee 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 five additional commits since the last revision:
> 
>  - Merge branch 'master' into 8343855
>  - Merge branch 'master' into 8343855
>  - Merge branch 'master' into 8343855
>  - Revert part of 8309118
>  - Backport bd6152f5967107d7b32db9bcfa224fc07314f098

Merged with master and seems fine now.

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

PR Comment: https://git.openjdk.org/jdk21u-dev/pull/1585#issuecomment-2809967438