RFR: 8310106: sun.security.ssl.SSLHandshake.getHandshakeProducer() incorrectly checks handshakeConsumers

Jaikiran Pai jpai at openjdk.org
Fri Jun 16 05:41:20 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.

Thank you Brad and Xuelei for the reviews.

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

PR Comment: https://git.openjdk.org/jdk/pull/14483#issuecomment-1594132694



More information about the security-dev mailing list