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