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