RFR: 8309214: sun/security/pkcs11/KeyStore/CertChainRemoval.java fails after 8301154
Jamil Nimeh
jnimeh at openjdk.org
Tue Aug 22 17:25:17 UTC 2023
On Thu, 3 Aug 2023 20:51:33 GMT, Valerie Peng <valeriep at openjdk.org> wrote:
> This change addresses the scenario where a certificate is first stored as part of a certificate chain and then stored again as a certificate corresponding to a PrivateKey entry. Newer version of NSS errors out with CKR_GENERAL_ERROR with the 2nd store, i.e. C_CreateObject() call.
>
> Proposed fix is to check for match before calling C_CreateObject(), if a match is found, set its alias instead.
This looks good. Does the bug need a noreg label since this is addressing an existing test failure?
-------------
Marked as reviewed by jnimeh (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/15146#pullrequestreview-1589954528
More information about the security-dev
mailing list