13053eadb0
- Added nss-bmo1930797.patch to fix failing tests in testsuite
- update to NSS 3.106
* bmo#1925975 - NSS 3.106 should be distributed with NSPR 4.36.
* bmo#1923767 - pk12util: improve error handling in p12U_ReadPKCS12File.
* bmo#1899402 - Correctly destroy bulkkey in error scenario.
* bmo#1919997 - PKCS7 fuzz target, r=djackson,nss-reviewers.
* bmo#1923002 - Extract certificates with handshake collection script.
* bmo#1923006 - Specify len_control for fuzz targets.
* bmo#1923280 - Fix memory leak in dumpCertificatePEM.
* bmo#1102981 - Fix UBSan errors for SECU_PrintCertificate and
SECU_PrintCertificateBasicInfo.
* bmo#1921528 - add new error codes to mozilla::pkix for Firefox to use.
* bmo#1921768 - allow null phKey in NSC_DeriveKey.
* bmo#1921801 - Only create seed corpus zip from existing corpus.
* bmo#1826035 - Use explicit allowlist for for KDF PRFS.
* bmo#1920138 - Increase optimization level for fuzz builds.
* bmo#1920470 - Remove incorrect assert.
* bmo#1914870 - Use libFuzzer options from fuzz/options/\*.options in CI.
* bmo#1920945 - Polish corpus collection for automation.
* bmo#1917572 - Detect new and unfuzzed SSL options.
* bmo#1804646 - PKCS12 fuzzing target.
- requires NSPR 4.36
OBS-URL: https://build.opensuse.org/package/show/mozilla:Factory/mozilla-nss?expand=0&rev=465
128 lines
4.4 KiB
Diff
128 lines
4.4 KiB
Diff
# HG changeset patch
|
|
# Parent 5786c2bb5c229b530e95e435ee0cf51314359e7b
|
|
|
|
Index: nss/lib/softoken/pkcs11c.c
|
|
===================================================================
|
|
--- nss.orig/lib/softoken/pkcs11c.c
|
|
+++ nss/lib/softoken/pkcs11c.c
|
|
@@ -5132,6 +5132,88 @@ pairwise_signverify_mech (CK_SESSION_HAN
|
|
return crv;
|
|
}
|
|
|
|
+/* This function regenerates a public key from a private key
|
|
+ * (not simply returning the saved public key) and compares it
|
|
+ * to the given publicKey
|
|
+ */
|
|
+static CK_RV
|
|
+regeneratePublicKeyFromPrivateKeyAndCompare(NSSLOWKEYPrivateKey *currPrivKey,
|
|
+ NSSLOWKEYPublicKey *currPubKey)
|
|
+{
|
|
+ NSSLOWKEYPublicKey *pubk;
|
|
+ SECItem publicValue;
|
|
+ PLArenaPool *arena;
|
|
+
|
|
+ arena = PORT_NewArena(DER_DEFAULT_CHUNKSIZE);
|
|
+ if (arena == NULL) {
|
|
+ PORT_SetError(SEC_ERROR_NO_MEMORY);
|
|
+ return CKR_HOST_MEMORY;
|
|
+ }
|
|
+
|
|
+ switch (currPrivKey->keyType) {
|
|
+ case NSSLOWKEYDHKey:
|
|
+ pubk = (NSSLOWKEYPublicKey *)PORT_ArenaZAlloc(arena,
|
|
+ sizeof(NSSLOWKEYPublicKey));
|
|
+ if (pubk != NULL) {
|
|
+ SECStatus rv;
|
|
+
|
|
+ pubk->arena = arena;
|
|
+ pubk->keyType = currPrivKey->keyType;
|
|
+
|
|
+ // Regenerate the publicValue
|
|
+ rv = DH_Derive(&currPrivKey->u.dh.base, &currPrivKey->u.dh.prime,
|
|
+ &currPrivKey->u.dh.privateValue, &publicValue, 0);
|
|
+ if (rv != SECSuccess) {
|
|
+ break;
|
|
+ }
|
|
+ rv = SECITEM_CopyItem(arena, &pubk->u.dh.publicValue,
|
|
+ &publicValue);
|
|
+ SECITEM_ZfreeItem(&publicValue, PR_FALSE);
|
|
+ if (rv != SECSuccess) {
|
|
+ break;
|
|
+ }
|
|
+
|
|
+ if (SECITEM_CompareItem(&pubk->u.dh.publicValue, &currPubKey->u.dh.publicValue) != SECEqual) {
|
|
+ nsslowkey_DestroyPublicKey(pubk);
|
|
+ return CKR_GENERAL_ERROR;
|
|
+ }
|
|
+ nsslowkey_DestroyPublicKey(pubk);
|
|
+ return CKR_OK;
|
|
+ }
|
|
+ break;
|
|
+ case NSSLOWKEYECKey:
|
|
+ {
|
|
+ ECPrivateKey *privk = NULL;
|
|
+ SECStatus rv;
|
|
+
|
|
+ /* The "seed" is an octet stream corresponding to our private key.
|
|
+ * The new public key is derived from this + the parameters and
|
|
+ * stored in the new private key's publicValue. */
|
|
+ rv = EC_NewKeyFromSeed (&currPrivKey->u.ec.ecParams,
|
|
+ &privk,
|
|
+ currPrivKey->u.ec.privateValue.data,
|
|
+ currPrivKey->u.ec.privateValue.len);
|
|
+ if (rv != SECSuccess)
|
|
+ break;
|
|
+
|
|
+ /* Verify that the passed-in public value is equal to the one derived */
|
|
+ if (SECITEM_CompareItem (&privk->publicValue, &currPubKey->u.ec.publicValue) != SECEqual) {
|
|
+ PORT_FreeArena (privk->ecParams.arena, PR_TRUE);
|
|
+ return CKR_GENERAL_ERROR;
|
|
+ }
|
|
+
|
|
+ PORT_FreeArena (privk->ecParams.arena, PR_TRUE);
|
|
+ return CKR_OK;
|
|
+ }
|
|
+ break;
|
|
+ default:
|
|
+ break;
|
|
+ }
|
|
+
|
|
+ PORT_FreeArena(arena, PR_TRUE);
|
|
+ return CKR_GENERAL_ERROR;
|
|
+}
|
|
+
|
|
/*
|
|
* FIPS 140-2 pairwise consistency check utilized to validate key pair.
|
|
*
|
|
@@ -5484,6 +5566,30 @@ sftk_PairwiseConsistencyCheck(CK_SESSION
|
|
}
|
|
}
|
|
|
|
+ // Regenerate the publicKey from the privateKey and compare it to the
|
|
+ // original publicKey
|
|
+ if (keyType == CKK_DH || keyType == CKK_EC) {
|
|
+ NSSLOWKEYPrivateKey *currPrivKey = sftk_GetPrivKey(privateKey, CKK_DH, &crv);
|
|
+ if (crv != CKR_OK) {
|
|
+ return crv;
|
|
+ }
|
|
+ if (!currPrivKey) {
|
|
+ return CKR_DEVICE_ERROR;
|
|
+ }
|
|
+
|
|
+ NSSLOWKEYPublicKey *currPubKey = sftk_GetPubKey(publicKey, CKK_DH, &crv);
|
|
+ if (crv != CKR_OK) {
|
|
+ return crv;
|
|
+ }
|
|
+ if (!currPubKey) {
|
|
+ return CKR_DEVICE_ERROR;
|
|
+ }
|
|
+
|
|
+ crv = regeneratePublicKeyFromPrivateKeyAndCompare(currPrivKey, currPubKey);
|
|
+ if (crv != CKR_OK) {
|
|
+ return crv;
|
|
+ }
|
|
+ }
|
|
return CKR_OK;
|
|
}
|
|
|