Looks fine to me.

Just two minor comments.  The run tag in the test may be not necessary.
Like EC algorithm, maybe the PKCS11 implementation of RSA and DSA
algorithms can also be checked on some platform if not using provider
option.

+      main0("RSA", 2048, "SHA256withRSA", null);
+      main0("DSA", 2048, "SHA256withDSA", null);

Xuelei

On 7/20/2016 3:10 AM, Vincent Ryan wrote:
> Please review this fix to apply stricter length checks when verifying public 
> key signatures.
> Thanks.
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8161571
> Webrev: http://cr.openjdk.java.net/~vinnie/8161571/webrev.00/
> 

Reply via email to