Looks fine to me. Thanks, Xuelei
On 6/15/2016 5:37 AM, joe darcy wrote: > The test > > javax/net/ssl/TLSv12/ShortRSAKey512.java > > has been seen to intermittently fail (JDK-8159501). The test should be > marked accordingly. > > Please review the patch below which does this. > > Thanks, > > -Joe > > diff -r d6a1ad87842f test/javax/net/ssl/TLSv12/ShortRSAKey512.java > --- a/test/javax/net/ssl/TLSv12/ShortRSAKey512.java Tue Jun 14 > 09:03:12 2016 -0700 > +++ b/test/javax/net/ssl/TLSv12/ShortRSAKey512.java Tue Jun 14 > 14:36:39 2016 -0700 > @@ -1,5 +1,5 @@ > /* > - * Copyright (c) 2012, 2015, Oracle and/or its affiliates. All rights > reserved. > + * Copyright (c) 2012, 2016, Oracle and/or its affiliates. All rights > reserved. > * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. > * > * This code is free software; you can redistribute it and/or modify it > @@ -33,6 +33,7 @@ > * > * SunJSSE does not support dynamic system properties, no way to > re-use > * system properties in samevm/agentvm mode. > + * @key intermittent > * @run main/othervm ShortRSAKey512 PKIX > * @run main/othervm ShortRSAKey512 SunX509 > */ >