This change looks fine.

Tony

On 3/7/19 4:58 PM, Valerie Peng wrote:

Anyone has time to help review this fix? According to the bug report, some vendors require the keytype attribute to match the encryption algorithm. So, the change is to match the key type based on the algorithm specified. There is no regression test as this issue is not reproducible against Solaris or NSS PKCS11 libraries.

Bug: https://bugs.openjdk.java.net/browse/JDK-8213008

Webrev: http://cr.openjdk.java.net/~valeriep/8213008/webrev.00/

Thanks,

Valerie


Reply via email to