RFR: 8274736: Concurrent read/close of SSLSockets causes SSLSessions to be invalidated unnecessarily [v4]

Bradford Wetmore wetmore at openjdk.java.net
Fri Nov 5 22:33:46 UTC 2021


On Fri, 5 Nov 2021 17:23:40 GMT, Jamil Nimeh <jnimeh at openjdk.org> wrote:

>> This fix handles cases where two or more threads may be acting on a single SSLSocket instance.  In cases where one thread closes the socket while the other is in the middle of a read, a SocketException is appropriately thrown, but it results in SSLSession invalidation even though the session was legitimately negotiated at an earlier point in time and should remain in the cache.
>> 
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8274736
>
> Jamil Nimeh has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Do not reinstantiate SSLContext, add more iterations, add test batch without using session tickets

test/jdk/sun/security/ssl/SSLSessionImpl/NoInvalidateSocketException.java line 34:

> 32:  * @test
> 33:  * @bug 8274736
> 34:  * @summary Concurrent read/close of SSLSockets causes SSLSessions to be invalidated unnecessarily

Minor nit.  Can you keep these under 80 chars?

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

PR: https://git.openjdk.java.net/jdk/pull/6197



More information about the security-dev mailing list