[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