On Sat, 4 Jan 2025 01:18:01 GMT, Valerie Peng <[email protected]> wrote:
>> Martin Balao has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Check disabled PKCS #11 mechanisms when concatenating keys and data. >> >> Co-authored-by: Martin Balao Alonso <[email protected]> >> Co-authored-by: Francisco Ferrari Bihurriet <[email protected]> > > src/jdk.crypto.cryptoki/share/classes/sun/security/pkcs11/P11KDF.java line 45: > >> 43: private final Token token; >> 44: private final P11SecretKeyFactory.HKDFKeyInfo svcKi; >> 45: private final long hmacMechanism; > > `svcki` and `hmacMechanism` are HKDF specific, so maybe this class should be > named `P11HKDF` instead? Our original intention was to keep it more open but it's true that in the end it's HKDF-specific. I'll rename it. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/22215#discussion_r1905965613
