RFR: 8314063 : The socket is not closed in Connection::createSocket when the handshake failed for LDAP connection [v2]

[Weibing Xiao]

On Wed, 16 Aug 2023 22:49:24 GMT, Mark Sheppard <msheppar at openjdk.org> wrote:

>> Weibing Xiao has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   update the test code
>
> test/jdk/com/sun/jndi/ldap/LdapSSLHandshakeFailureTest.java line 173:
> 
>> 171:         public void run() {
>> 172:             try (Socket socket = serverSocket.accept()) {
>> 173:                 Thread.sleep(10000);
> 
> What's the purpose of the sleep ? 
> Regardless, based on the test semantics alluded in the test name, the server should never enter the read block. So is this code redundant? Or is it there just in case the accept and the handshake succeeds?

It will slow down the handshake process after the socket is created and connected with the server. The bug is showing the handshake failure and the socket is not closed properly.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15294#discussion_r1296522377