RFR: 8294985: SSLEngine throws IAE during parsing of X500Principal [v24]

[Sean Mullan]

On Tue, 30 May 2023 14:59:11 GMT, Kevin Driver <kdriver at openjdk.org> wrote:

>> Fixes: [JDK-8294985](https://bugs.openjdk.org/browse/JDK-8294985)
>
> 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 26 additional commits since the last revision:
> 
>  - Merge branch 'master' of github.com:openjdk/jdk into JDK-8294985
>  - wrapping
>  - addressed remaining code review comments
>  - Merge branch 'master' of github.com:openjdk/jdk into JDK-8294985
>  - run tests with othervm
>  - working testcase for CertificateRequest for TLS1.2
>  - new version of second test
>  - initial commit of second test
>  - reintroduce bug id to test header
>  - Merge remote-tracking branch 'upstream/master' into JDK-8294985
>  - ... and 16 more: https://git.openjdk.org/jdk/compare/38960d7b...2a35a7cc

src/java.base/share/classes/sun/security/ssl/CertificateAuthoritiesExtension.java line 28:

> 26: package sun.security.ssl;
> 27: 
> 28: import sun.security.ssl.SSLExtension.ExtensionConsumer;

Please keep the previous import order for consistency with other code in this package. All code in `sun.security.ssl` generally orders the imports as `java`, `javax`, then internal (`sun`), etc.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13466#discussion_r1210451045