Code Review Request, 8144313, Test SessionTimeOutTests can be timeout
Sean Mullan
sean.mullan at oracle.com
Tue Dec 1 12:36:28 UTC 2015
serverReady doesn't need to be volatile anymore.
Looks good otherwise.
--Sean
On 12/01/2015 05:48 AM, Xuelei Fan wrote:
> Hi,
>
> Please review this test update:
>
> http://cr.openjdk.java.net/~xuelei/8144313/webrev.00/
>
> In test/javax/net/ssl/SSLSession/SessionTimeOutTests.java, the update of
> "serverReady" variable was not synchronized when performing multiple
> operations, as may result that the variable cannot be calculated
> correctly, and result in timeout accordingly.
>
> Update to use AtomicInteger for the atomic update.
>
> Thanks,
> Xuelei
>
More information about the security-dev
mailing list