RFR: 8310106: sun.security.ssl.SSLHandshake.getHandshakeProducer() incorrectly checks handshakeConsumers
Xue-Lei Andrew Fan
xuelei at openjdk.org
Fri Jun 16 02:56:15 UTC 2023
On Thu, 15 Jun 2023 06:02:13 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:
> Can I please get a review of this change which fixes the issue noted in https://bugs.openjdk.org/browse/JDK-8310106?
>
> The commit here fixes the typo in `sun.security.ssl.SSLHandshake.getHandshakeProducer()` method to correctly use `handshakeProducers`.
>
> I checked if a new jtreg test could be added for this, but given where this change resides and the nature of this change, I don't see a way to add one. Existing tests in `tier1`, `tier2` and `tier3` continue to pass with this change.
Nice catch. Thanks!
-------------
Marked as reviewed by xuelei (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/14483#pullrequestreview-1482644422
More information about the security-dev
mailing list