RFR: 8290005: com/sun/jndi/ldap/LdapCBPropertiesTest.java failling with NullPointerException [v4]

Fri Jun 16 14:56:10 UTC 2023

On Fri, 16 Jun 2023 14:45:52 GMT, Matthew Donovan <mdonovan at openjdk.org> wrote:

>> I may update both SSLSocketImpl and SSLEngineImpl, as SSLSocketImpl uses the locks similar to SSLEngineImpl.
>
> There is a difference, though. The close() method in SSLSocketImpl is not synchronized and there is the chance of a NullPointerException in duplexCloseOutput() because `conContext.handshakeContext` is being set to null by the TransportContext class in a different thread. I think that can affect any method in SSLSocketImpl that accesses that field.

Yes, socket close is a headache problem for me.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13742#discussion_r1232359603