An interesting bug.  Looks fine to me.

Xuelei

On 1/1/2016 1:26 AM, Wang Weijun wrote:
> Hi
> 
> Please review the fix of https://bugs.openjdk.java.net/browse/JDK-8146377. 
> Some years have more days than 365 and the test fails. The fix is simply
> 
> diff --git a/test/sun/security/tools/jarsigner/concise_jarsigner.sh 
> b/test/sun/security/tools/jarsigner/concise_jarsigner.sh
> --- a/test/sun/security/tools/jarsigner/concise_jarsigner.sh
> +++ b/test/sun/security/tools/jarsigner/concise_jarsigner.sh
> @@ -69,8 +69,8 @@
>  # First part: output format
>  # ==========================================================
> 
> -$KT -genkeypair -alias a1 -dname CN=a1 -validity 365
> -$KT -genkeypair -alias a2 -dname CN=a2 -validity 365
> +$KT -genkeypair -alias a1 -dname CN=a1 -validity 366
> +$KT -genkeypair -alias a2 -dname CN=a2 -validity 366
> 
>  # a.jar includes 8 unsigned, 2 signed by a1 and a2, 2 signed by a3
>  $JAR cvf a.jar A1.class A2.class
> 
> Thanks
> Max
> 

Reply via email to