RFR: 8294985: SSLEngine throws IAE during parsing of X500Principal [v19]
Kevin Driver
kdriver at openjdk.org
Fri May 26 21:39:32 UTC 2023
On Tue, 23 May 2023 18:15:57 GMT, Xue-Lei Andrew Fan <xuelei at openjdk.org> wrote:
>> Kevin Driver 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 17 additional commits since the last revision:
>>
>> - Merge remote-tracking branch 'upstream/master' into JDK-8294985
>> - additional code review comments
>> - rename class and remove bug id from test header
>> - removing block that isn't reached
>> - fix bug id in test header
>> - reworked example into a jtreg test
>> - whitespace adjustments
>> - all review comments applied
>> - optimize imports and change toString
>> - review comments addressed
>> - ... and 7 more: https://git.openjdk.org/jdk/compare/47d94ef0...d9f0c667
>
> test/jdk/sun/security/ssl/SSLEngineImpl/TestBadDNForPeerCA.java line 29:
>
>> 27: * @summary verify correct exception handling in the event of an unparseable
>> 28: * DN in the peer CA
>> 29: */
>
> Please run tests for JSSE/TLS in othervm mode. Otherwise, the behavior may be impacted with each other, and the failure debugging is pretty hard. For an example, please refer to test/jdk/javax/net/ssl/SSLEngine/IllegalHandshakeMessage.java
See: https://github.com/openjdk/jdk/pull/13466/commits/67506a21231793df250e08ea090e6c3461d7b4f8
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13466#discussion_r1207388554
More information about the security-dev
mailing list