RFR: 8339403: sun.security.ssl.StatusResponseManager.get swallows interrupt status [v2]

Jamil Nimeh jnimeh at openjdk.org
Wed Oct 2 22:53:01 UTC 2024


> This PR corrects a flaw in the StatusResponseManager where it was incorrectly swallowing the interrupt status when either an invokeAll was called (spawning the threads to fetch each OCSP response) or when attempting to grab the data from one of the Futures returned from the fetches.
> Additionally, I made a small change that, in the unlikely event of a non-IOException being thrown, only that specific fetch is affected, but other successful fetches on different threads can complete and can be added to the resulting responseMap.

Jamil Nimeh 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 three additional commits since the last revision:

 - Add continue at end of ExecutionException catch block
 - Merge with main
 - 8339403: sun.security.ssl.StatusResponseManager.get swallows interrupt status

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/21230/files
  - new: https://git.openjdk.org/jdk/pull/21230/files/386d7dc2..b2990b84

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=21230&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=21230&range=00-01

  Stats: 215783 lines in 2337 files changed: 187619 ins; 15481 del; 12683 mod
  Patch: https://git.openjdk.org/jdk/pull/21230.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/21230/head:pull/21230

PR: https://git.openjdk.org/jdk/pull/21230


More information about the security-dev mailing list