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