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

Paul Hohensee phh at openjdk.org
Wed Apr 16 15:01:10 UTC 2025


> 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

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

Changes:
  - all: https://git.openjdk.org/jdk21u-dev/pull/1585/files
  - new: https://git.openjdk.org/jdk21u-dev/pull/1585/files/f4e41e17..eaffe28d

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk21u-dev&pr=1585&range=04
 - incr: https://webrevs.openjdk.org/?repo=jdk21u-dev&pr=1585&range=03-04

  Stats: 18050 lines in 233 files changed: 17294 ins; 373 del; 383 mod
  Patch: https://git.openjdk.org/jdk21u-dev/pull/1585.diff
  Fetch: git fetch https://git.openjdk.org/jdk21u-dev.git pull/1585/head:pull/1585

PR: https://git.openjdk.org/jdk21u-dev/pull/1585


More information about the jdk-updates-dev mailing list