Looks fine to me. Thanks, Xuelei
On 3/8/2017 1:20 PM, Amanda Jiang wrote:
Hi All, Could you please help to review the patch below? sun/security/ssl/X509KeyManager/PreferredKey.java failed on all platforms since related certificate in keystore has been expired. To fix the bug, the keystore should be carefully updated with new certificate since a lot of tests depended on the keystore. Temporarily add this test to problem list during fixing it. Bug: https://bugs.openjdk.java.net/browse/JDK-8176400 diff -r 4f151e0530e6 test/ProblemList.txt --- a/test/ProblemList.txt Wed Mar 08 12:14:25 2017 -0800 +++ b/test/ProblemList.txt Wed Mar 08 13:17:06 2017 -0800 @@ -219,6 +219,7 @@ sun/security/krb5/auto/Basic.java 8176296 generic-all +sun/security/ssl/X509KeyManager/PreferredKey.java 8176354 generic-all ############################################################################ # jdk_sound Thanks, Amanda