[JDK 8]Code review request 8022228, Intermittent test failures in sun/security/ssl/javax/net/ssl/NewAPIs
Weijun Wang
weijun.wang at oracle.com
Thu Aug 22 01:30:08 UTC 2013
SessionTimeOutTests.java:
411 local.initCause(remote);
What if local already has a cause? Will this overwrite it?
427 exception.addSuppressed(startException);
Is it possible startException is null?
Same for the other test.
Also, if you have to do all these checks in every SSL test, can they be
added into a library?
Thanks
Max
On 8/22/13 9:18 AM, Xuelei Fan wrote:
> Hi Weijun,
>
> Please review this regression test bug fix for JDK 8.
>
> Webrev: http://cr.openjdk.java.net/~xuelei/8022228/webrev.00/
>
> The timeout issue of SessionTimeOutTests.java is still unknown. This
> fix is just try to use new SSL testing template so as to expose more
> information.
>
> The issue of SessionCacheSizeTests is that the client side may kick off
> a connection before server is ready to listen on a particular port.
>
> Thanks,
> Xuelei
>
More information about the security-dev
mailing list