RFR: 8264849: Add KW and KWP support to PKCS11 provider

Anthony Scarpino ascarpino at openjdk.java.net
Fri Oct 1 00:55:27 UTC 2021


On Fri, 17 Sep 2021 23:22:21 GMT, Valerie Peng <valeriep at openjdk.org> wrote:

> Anyone has time to review this RFE for adding AES cipher with KW, KWP modes support to SunPKCS11 provider?
> 
> The main changes are in only one new class, i.e. P11KeyWrapCipher.java, which is the CipherSpi impl for the native PKCS11 key wrap mechanisms. When testing against NSS library, it seems that they only support the single part enc/dec PKCS11 APIs, so have to use a new class as existing P11Cipher class relies on the multi part enc/dec PKCS11 APIs and do not support key wrapping/unwrapping.
> 
> The rest are minor code refactoring and updates for the PKCS11 Exception class.
> The new regression tests are adapted from existing key wrap regression tests for SunJCE provider.
> 
> Thanks,
> Valerie

test/jdk/sun/security/pkcs11/Cipher/KeyWrap/TestGeneral.java line 30:

> 28:  *     AES/KW/PKCS5Padding, and AES/KWP/NoPadding impls of SunPKCS11 provider.
> 29:  * @library /test/lib ../..
> 30:  * @run main/othervm TestGeneral

General question about all the tests.  They use othervm, did they not work with agentvm?

test/jdk/sun/security/pkcs11/Cipher/KeyWrap/TestGeneral.java line 74:

> 72:                 }
> 73: 
> 74:                 //System.out.println("enc out.length=" + out.length);

Lelftover debug line?

-------------

PR: https://git.openjdk.java.net/jdk/pull/5569



More information about the security-dev mailing list