RFR: 8339403: sun.security.ssl.StatusResponseManager.get swallows interrupt status
Valerie Peng
valeriep at openjdk.org
Wed Oct 2 21:40:36 UTC 2024
On Fri, 27 Sep 2024 18:48:23 GMT, Jamil Nimeh <jnimeh at openjdk.org> wrote:
> 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.
Looks good to me. Thanks~
-------------
Marked as reviewed by valeriep (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/21230#pullrequestreview-2344147380
More information about the security-dev
mailing list