Blame SOURCES/nss-8-fix-public-key-from-priv.patch

3d912a
diff -up ./gtests/pk11_gtest/pk11_import_unittest.cc.pub-priv-mech ./gtests/pk11_gtest/pk11_import_unittest.cc
3d912a
--- ./gtests/pk11_gtest/pk11_import_unittest.cc.pub-priv-mech	2019-05-10 14:14:18.000000000 -0700
3d912a
+++ ./gtests/pk11_gtest/pk11_import_unittest.cc	2019-06-05 16:43:42.276498676 -0700
3d912a
@@ -78,17 +78,40 @@ class Pk11KeyImportTestBase : public ::t
3d912a
   CK_MECHANISM_TYPE mech_;
3d912a
 
3d912a
  private:
3d912a
+  SECItem GetPublicComponent(ScopedSECKEYPublicKey& pub_key) {
3d912a
+    SECItem null = { siBuffer, NULL, 0};
3d912a
+    switch(SECKEY_GetPublicKeyType(pub_key.get())) {
3d912a
+    case rsaKey:
3d912a
+    case rsaPssKey:
3d912a
+    case rsaOaepKey:
3d912a
+       return pub_key->u.rsa.modulus;
3d912a
+    case keaKey:
3d912a
+       return pub_key->u.kea.publicValue;
3d912a
+    case dsaKey:
3d912a
+       return pub_key->u.dsa.publicValue;
3d912a
+    case dhKey:
3d912a
+       return pub_key->u.dh.publicValue;
3d912a
+    case ecKey:
3d912a
+       return pub_key->u.ec.publicValue;
3d912a
+    case fortezzaKey: /* depricated */
3d912a
+    case nullKey:
3d912a
+    /* didn't use default here so we can catch new key types at compile time */
3d912a
+       break;
3d912a
+    }
3d912a
+    return null;
3d912a
+  }
3d912a
   void CheckForPublicKey(const ScopedSECKEYPrivateKey& priv_key,
3d912a
                          const SECItem* expected_public) {
3d912a
     // Verify the public key exists.
3d912a
     StackSECItem priv_id;
3d912a
+    KeyType type = SECKEY_GetPrivateKeyType(priv_key.get());
3d912a
     SECStatus rv = PK11_ReadRawAttribute(PK11_TypePrivKey, priv_key.get(),
3d912a
                                          CKA_ID, &priv_id);
3d912a
     ASSERT_EQ(SECSuccess, rv) << "Couldn't read CKA_ID from private key: "
3d912a
                               << PORT_ErrorToName(PORT_GetError());
3d912a
 
3d912a
     CK_ATTRIBUTE_TYPE value_type = CKA_VALUE;
3d912a
-    switch (SECKEY_GetPrivateKeyType(priv_key.get())) {
3d912a
+    switch (type) {
3d912a
       case rsaKey:
3d912a
         value_type = CKA_MODULUS;
3d912a
         break;
3d912a
@@ -106,6 +129,8 @@ class Pk11KeyImportTestBase : public ::t
3d912a
         FAIL() << "unknown key type";
3d912a
     }
3d912a
 
3d912a
+    // Scan public key objects until we find one with the same CKA_ID as
3d912a
+    // priv_key
3d912a
     std::unique_ptr<PK11GenericObject, PK11GenericObjectsDeleter> objs(
3d912a
         PK11_FindGenericObjects(slot_.get(), CKO_PUBLIC_KEY));
3d912a
     ASSERT_NE(nullptr, objs);
3d912a
@@ -128,20 +153,46 @@ class Pk11KeyImportTestBase : public ::t
3d912a
       ASSERT_EQ(1U, token.len);
3d912a
       ASSERT_NE(0, token.data[0]);
3d912a
 
3d912a
-      StackSECItem value;
3d912a
-      rv = PK11_ReadRawAttribute(PK11_TypeGeneric, obj, value_type, &value);
3d912a
+      StackSECItem raw_value;
3d912a
+      SECItem decoded_value;
3d912a
+      rv = PK11_ReadRawAttribute(PK11_TypeGeneric, obj, value_type, &raw_value);
3d912a
       ASSERT_EQ(SECSuccess, rv);
3d912a
+      SECItem value = raw_value;
3d912a
 
3d912a
+      // Decode the EC_POINT and check the output against expected.
3d912a
       // CKA_EC_POINT isn't stable, see Bug 1520649.
3d912a
+      ScopedPLArenaPool arena(PORT_NewArena(DER_DEFAULT_CHUNKSIZE));
3d912a
+      ASSERT_TRUE(arena);
3d912a
       if (value_type == CKA_EC_POINT) {
3d912a
-        continue;
3d912a
-      }
3d912a
 
3d912a
+        // If this fails due to the noted inconsistency, we may need to
3d912a
+        // check the whole raw_value, or remove a leading UNCOMPRESSED_POINT tag
3d912a
+        rv = SEC_QuickDERDecodeItem(arena.get(), &decoded_value,
3d912a
+                                    SEC_ASN1_GET(SEC_OctetStringTemplate),
3d912a
+                                    &raw_value);
3d912a
+        ASSERT_EQ(SECSuccess, rv);
3d912a
+        value = decoded_value;
3d912a
+      }
3d912a
       ASSERT_TRUE(SECITEM_ItemsAreEqual(expected_public, &value))
3d912a
           << "expected: "
3d912a
           << DataBuffer(expected_public->data, expected_public->len)
3d912a
           << std::endl
3d912a
           << "actual: " << DataBuffer(value.data, value.len) << std::endl;
3d912a
+
3d912a
+      // Finally, convert the private to public and ensure it matches.
3d912a
+      ScopedSECKEYPublicKey pub_key(
3d912a
+            SECKEY_ConvertToPublicKey(priv_key.get()));
3d912a
+      ASSERT_TRUE(pub_key);
3d912a
+      SECItem converted_public = GetPublicComponent(pub_key);
3d912a
+      ASSERT_TRUE(converted_public.len != 0);
3d912a
+
3d912a
+      ASSERT_TRUE(SECITEM_ItemsAreEqual(expected_public, &converted_public))
3d912a
+            << "expected: "
3d912a
+            << DataBuffer(expected_public->data, expected_public->len)
3d912a
+            << std::endl
3d912a
+            << "actual: "
3d912a
+            << DataBuffer(converted_public.data, converted_public.len)
3d912a
+            << std::endl;
3d912a
     }
3d912a
   }
3d912a
 
3d912a
diff -up ./lib/cryptohi/seckey.c.pub-priv-mech ./lib/cryptohi/seckey.c
3d912a
--- ./lib/cryptohi/seckey.c.pub-priv-mech	2019-05-10 14:14:18.000000000 -0700
3d912a
+++ ./lib/cryptohi/seckey.c	2019-06-05 16:43:42.277498676 -0700
3d912a
@@ -1206,6 +1206,37 @@ SECKEY_CopyPublicKey(const SECKEYPublicK
3d912a
     return NULL;
3d912a
 }
3d912a
 
3d912a
+/*
3d912a
+ * Use the private key to find a public key handle. The handle will be on
3d912a
+ * the same slot as the private key.
3d912a
+ */
3d912a
+static CK_OBJECT_HANDLE
3d912a
+seckey_FindPublicKeyHandle(SECKEYPrivateKey *privk, SECKEYPublicKey *pubk)
3d912a
+{
3d912a
+    CK_OBJECT_HANDLE keyID;
3d912a
+
3d912a
+    /* this helper function is only used below. If we want to make this more
3d912a
+     * general, we would need to free up any already cached handles if the
3d912a
+     * slot doesn't match up with the private key slot */
3d912a
+    PORT_Assert(pubk->pkcs11ID == CK_INVALID_HANDLE);
3d912a
+
3d912a
+    /* first look for a matching public key */
3d912a
+    keyID = PK11_MatchItem(privk->pkcs11Slot, privk->pkcs11ID, CKO_PUBLIC_KEY);
3d912a
+    if (keyID != CK_INVALID_HANDLE) {
3d912a
+        return keyID;
3d912a
+    }
3d912a
+
3d912a
+    /* none found, create a temp one, make the pubk the owner */
3d912a
+    pubk->pkcs11ID = PK11_DerivePubKeyFromPrivKey(privk);
3d912a
+    if (pubk->pkcs11ID == CK_INVALID_HANDLE) {
3d912a
+        /* end of the road. Token doesn't have matching public key, nor can
3d912a
+          * token regenerate a new public key from and existing private key. */
3d912a
+        return CK_INVALID_HANDLE;
3d912a
+    }
3d912a
+    pubk->pkcs11Slot = PK11_ReferenceSlot(privk->pkcs11Slot);
3d912a
+    return pubk->pkcs11ID;
3d912a
+}
3d912a
+
3d912a
 SECKEYPublicKey *
3d912a
 SECKEY_ConvertToPublicKey(SECKEYPrivateKey *privk)
3d912a
 {
3d912a
@@ -1213,6 +1244,8 @@ SECKEY_ConvertToPublicKey(SECKEYPrivateK
3d912a
     PLArenaPool *arena;
3d912a
     CERTCertificate *cert;
3d912a
     SECStatus rv;
3d912a
+    CK_OBJECT_HANDLE pubKeyHandle;
3d912a
+    SECItem decodedPoint;
3d912a
 
3d912a
     /*
3d912a
      * First try to look up the cert.
3d912a
@@ -1243,11 +1276,47 @@ SECKEY_ConvertToPublicKey(SECKEYPrivateK
3d912a
 
3d912a
     switch (privk->keyType) {
3d912a
         case nullKey:
3d912a
-        case dhKey:
3d912a
-        case dsaKey:
3d912a
             /* Nothing to query, if the cert isn't there, we're done -- no way
3d912a
              * to get the public key */
3d912a
             break;
3d912a
+        case dsaKey:
3d912a
+            pubKeyHandle = seckey_FindPublicKeyHandle(privk, pubk);
3d912a
+            if (pubKeyHandle == CK_INVALID_HANDLE)
3d912a
+                break;
3d912a
+            rv = PK11_ReadAttribute(privk->pkcs11Slot, pubKeyHandle,
3d912a
+                                    CKA_BASE, arena, &pubk->u.dsa.params.base);
3d912a
+            if (rv != SECSuccess)
3d912a
+                break;
3d912a
+            rv = PK11_ReadAttribute(privk->pkcs11Slot, pubKeyHandle,
3d912a
+                                    CKA_PRIME, arena, &pubk->u.dsa.params.prime);
3d912a
+            if (rv != SECSuccess)
3d912a
+                break;
3d912a
+            rv = PK11_ReadAttribute(privk->pkcs11Slot, pubKeyHandle,
3d912a
+                                    CKA_SUBPRIME, arena, &pubk->u.dsa.params.subPrime);
3d912a
+            if (rv != SECSuccess)
3d912a
+                break;
3d912a
+            rv = PK11_ReadAttribute(privk->pkcs11Slot, pubKeyHandle,
3d912a
+                                    CKA_VALUE, arena, &pubk->u.dsa.publicValue);
3d912a
+            if (rv != SECSuccess)
3d912a
+                break;
3d912a
+            return pubk;
3d912a
+        case dhKey:
3d912a
+            pubKeyHandle = seckey_FindPublicKeyHandle(privk, pubk);
3d912a
+            if (pubKeyHandle == CK_INVALID_HANDLE)
3d912a
+                break;
3d912a
+            rv = PK11_ReadAttribute(privk->pkcs11Slot, pubKeyHandle,
3d912a
+                                    CKA_BASE, arena, &pubk->u.dh.base);
3d912a
+            if (rv != SECSuccess)
3d912a
+                break;
3d912a
+            rv = PK11_ReadAttribute(privk->pkcs11Slot, pubKeyHandle,
3d912a
+                                    CKA_PRIME, arena, &pubk->u.dh.prime);
3d912a
+            if (rv != SECSuccess)
3d912a
+                break;
3d912a
+            rv = PK11_ReadAttribute(privk->pkcs11Slot, pubKeyHandle,
3d912a
+                                    CKA_VALUE, arena, &pubk->u.dh.publicValue);
3d912a
+            if (rv != SECSuccess)
3d912a
+                break;
3d912a
+            return pubk;
3d912a
         case rsaKey:
3d912a
             rv = PK11_ReadAttribute(privk->pkcs11Slot, privk->pkcs11ID,
3d912a
                                     CKA_MODULUS, arena, &pubk->u.rsa.modulus);
3d912a
@@ -1258,7 +1327,6 @@ SECKEY_ConvertToPublicKey(SECKEYPrivateK
3d912a
             if (rv != SECSuccess)
3d912a
                 break;
3d912a
             return pubk;
3d912a
-            break;
3d912a
         case ecKey:
3d912a
             rv = PK11_ReadAttribute(privk->pkcs11Slot, privk->pkcs11ID,
3d912a
                                     CKA_EC_PARAMS, arena, &pubk->u.ec.DEREncodedParams);
3d912a
@@ -1268,7 +1336,23 @@ SECKEY_ConvertToPublicKey(SECKEYPrivateK
3d912a
             rv = PK11_ReadAttribute(privk->pkcs11Slot, privk->pkcs11ID,
3d912a
                                     CKA_EC_POINT, arena, &pubk->u.ec.publicValue);
3d912a
             if (rv != SECSuccess || pubk->u.ec.publicValue.len == 0) {
3d912a
-                break;
3d912a
+                pubKeyHandle = seckey_FindPublicKeyHandle(privk, pubk);
3d912a
+                if (pubKeyHandle == CK_INVALID_HANDLE)
3d912a
+                    break;
3d912a
+                rv = PK11_ReadAttribute(privk->pkcs11Slot, pubKeyHandle,
3d912a
+                                        CKA_EC_POINT, arena, &pubk->u.ec.publicValue);
3d912a
+                if (rv != SECSuccess)
3d912a
+                    break;
3d912a
+            }
3d912a
+            /* ec.publicValue should be decoded, PKCS #11 defines CKA_EC_POINT
3d912a
+             * as encoded, but it's not always. try do decoded it and if it
3d912a
+             * succeeds store the decoded value */
3d912a
+            rv = SEC_QuickDERDecodeItem(arena, &decodedPoint,
3d912a
+                                        SEC_ASN1_GET(SEC_OctetStringTemplate), &pubk->u.ec.publicValue);
3d912a
+            if (rv == SECSuccess) {
3d912a
+                /* both values are in the public key arena, so it's safe to
3d912a
+                 * overwrite  the old value */
3d912a
+                pubk->u.ec.publicValue = decodedPoint;
3d912a
             }
3d912a
             pubk->u.ec.encoding = ECPoint_Undefined;
3d912a
             return pubk;
3d912a
@@ -1276,7 +1360,9 @@ SECKEY_ConvertToPublicKey(SECKEYPrivateK
3d912a
             break;
3d912a
     }
3d912a
 
3d912a
-    PORT_FreeArena(arena, PR_FALSE);
3d912a
+    /* must use Destroy public key here, because some paths create temporary
3d912a
+     * PKCS #11 objects which need to be freed */
3d912a
+    SECKEY_DestroyPublicKey(pubk);
3d912a
     return NULL;
3d912a
 }
3d912a
 
3d912a
diff -up ./lib/pk11wrap/pk11priv.h.pub-priv-mech ./lib/pk11wrap/pk11priv.h
3d912a
--- ./lib/pk11wrap/pk11priv.h.pub-priv-mech	2019-05-10 14:14:18.000000000 -0700
3d912a
+++ ./lib/pk11wrap/pk11priv.h	2019-06-05 16:43:42.277498676 -0700
3d912a
@@ -111,6 +111,7 @@ CK_OBJECT_HANDLE PK11_FindObjectForCert(
3d912a
 PK11SymKey *pk11_CopyToSlot(PK11SlotInfo *slot, CK_MECHANISM_TYPE type,
3d912a
                             CK_ATTRIBUTE_TYPE operation, PK11SymKey *symKey);
3d912a
 unsigned int pk11_GetPredefinedKeyLength(CK_KEY_TYPE keyType);
3d912a
+CK_OBJECT_HANDLE PK11_DerivePubKeyFromPrivKey(SECKEYPrivateKey *privKey);
3d912a
 
3d912a
 /**********************************************************************
3d912a
  *                   Certs
3d912a
diff -up ./lib/pk11wrap/pk11skey.c.pub-priv-mech ./lib/pk11wrap/pk11skey.c
3d912a
--- ./lib/pk11wrap/pk11skey.c.pub-priv-mech	2019-06-05 16:37:38.726685789 -0700
3d912a
+++ ./lib/pk11wrap/pk11skey.c	2019-06-05 16:43:42.278498675 -0700
3d912a
@@ -1841,6 +1841,35 @@ loser:
3d912a
 }
3d912a
 
3d912a
 /*
3d912a
+ * This regenerate a public key from a private key. This function is currently
3d912a
+ * NSS private. If we want to make it public, we need to add and optional
3d912a
+ * template or at least flags (a.la. PK11_DeriveWithFlags).
3d912a
+ */
3d912a
+CK_OBJECT_HANDLE
3d912a
+PK11_DerivePubKeyFromPrivKey(SECKEYPrivateKey *privKey)
3d912a
+{
3d912a
+    PK11SlotInfo *slot = privKey->pkcs11Slot;
3d912a
+    CK_MECHANISM mechanism;
3d912a
+    CK_OBJECT_HANDLE objectID = CK_INVALID_HANDLE;
3d912a
+    CK_RV crv;
3d912a
+
3d912a
+    mechanism.mechanism = CKM_NSS_PUB_FROM_PRIV;
3d912a
+    mechanism.pParameter = NULL;
3d912a
+    mechanism.ulParameterLen = 0;
3d912a
+
3d912a
+    PK11_EnterSlotMonitor(slot);
3d912a
+    crv = PK11_GETTAB(slot)->C_DeriveKey(slot->session, &mechanism,
3d912a
+                                         privKey->pkcs11ID, NULL, 0,
3d912a
+                                         &objectID);
3d912a
+    PK11_ExitSlotMonitor(slot);
3d912a
+    if (crv != CKR_OK) {
3d912a
+        PORT_SetError(PK11_MapError(crv));
3d912a
+        return CK_INVALID_HANDLE;
3d912a
+    }
3d912a
+    return objectID;
3d912a
+}
3d912a
+
3d912a
+/*
3d912a
  * This Generates a wrapping key based on a privateKey, publicKey, and two
3d912a
  * random numbers. For Mail usage RandomB should be NULL. In the Sender's
3d912a
  * case RandomA is generate, outherwize it is passed.
3d912a
diff -up ./lib/softoken/lowkey.c.pub-priv-mech ./lib/softoken/lowkey.c
3d912a
--- ./lib/softoken/lowkey.c.pub-priv-mech	2019-05-10 14:14:18.000000000 -0700
3d912a
+++ ./lib/softoken/lowkey.c	2019-06-05 16:44:20.469479019 -0700
3d912a
@@ -261,6 +261,7 @@ NSSLOWKEYPublicKey *
3d912a
 nsslowkey_ConvertToPublicKey(NSSLOWKEYPrivateKey *privk)
3d912a
 {
3d912a
     NSSLOWKEYPublicKey *pubk;
3d912a
+    SECItem publicValue;
3d912a
     PLArenaPool *arena;
3d912a
 
3d912a
     arena = PORT_NewArena(DER_DEFAULT_CHUNKSIZE);
3d912a
@@ -301,6 +302,19 @@ nsslowkey_ConvertToPublicKey(NSSLOWKEYPr
3d912a
 
3d912a
                 pubk->arena = arena;
3d912a
                 pubk->keyType = privk->keyType;
3d912a
+                /* if the public key value doesn't exist, calculate it */
3d912a
+                if (privk->u.dsa.publicValue.len == 0) {
3d912a
+                    rv = DH_Derive(&privk->u.dsa.params.base, &privk->u.dsa.params.prime,
3d912a
+                                   &privk->u.dsa.privateValue, &publicValue, 0);
3d912a
+                    if (rv != SECSuccess) {
3d912a
+                        break;
3d912a
+                    }
3d912a
+                    rv = SECITEM_CopyItem(privk->arena, &privk->u.dsa.publicValue, &publicValue);
3d912a
+                    SECITEM_FreeItem(&publicValue, PR_FALSE);
3d912a
+                    if (rv != SECSuccess) {
3d912a
+                        break;
3d912a
+                    }
3d912a
+                }
3d912a
                 rv = SECITEM_CopyItem(arena, &pubk->u.dsa.publicValue,
3d912a
                                       &privk->u.dsa.publicValue);
3d912a
                 if (rv != SECSuccess)
3d912a
@@ -327,6 +341,19 @@ nsslowkey_ConvertToPublicKey(NSSLOWKEYPr
3d912a
 
3d912a
                 pubk->arena = arena;
3d912a
                 pubk->keyType = privk->keyType;
3d912a
+                /* if the public key value doesn't exist, calculate it */
3d912a
+                if (privk->u.dh.publicValue.len == 0) {
3d912a
+                    rv = DH_Derive(&privk->u.dh.base, &privk->u.dh.prime,
3d912a
+                                   &privk->u.dh.privateValue, &publicValue, 0);
3d912a
+                    if (rv != SECSuccess) {
3d912a
+                        break;
3d912a
+                    }
3d912a
+                    rv = SECITEM_CopyItem(privk->arena, &privk->u.dh.publicValue, &publicValue);
3d912a
+                    SECITEM_FreeItem(&publicValue, PR_FALSE);
3d912a
+                    if (rv != SECSuccess) {
3d912a
+                        break;
3d912a
+                    }
3d912a
+                }
3d912a
                 rv = SECITEM_CopyItem(arena, &pubk->u.dh.publicValue,
3d912a
                                       &privk->u.dh.publicValue);
3d912a
                 if (rv != SECSuccess)
3d912a
diff -up ./lib/softoken/pkcs11c.c.pub-priv-mech ./lib/softoken/pkcs11c.c
3d912a
--- ./lib/softoken/pkcs11c.c.pub-priv-mech	2019-06-05 16:37:38.743685780 -0700
3d912a
+++ ./lib/softoken/pkcs11c.c	2019-06-05 16:44:20.472479017 -0700
3d912a
@@ -6569,6 +6569,10 @@ NSC_DeriveKey(CK_SESSION_HANDLE hSession
3d912a
             extractValue = PR_FALSE;
3d912a
             classType = CKO_PRIVATE_KEY;
3d912a
             break;
3d912a
+        case CKM_NSS_PUB_FROM_PRIV:
3d912a
+            extractValue = PR_FALSE;
3d912a
+            classType = CKO_PUBLIC_KEY;
3d912a
+            break;
3d912a
         case CKM_NSS_JPAKE_FINAL_SHA1:   /* fall through */
3d912a
         case CKM_NSS_JPAKE_FINAL_SHA256: /* fall through */
3d912a
         case CKM_NSS_JPAKE_FINAL_SHA384: /* fall through */
3d912a
@@ -6610,6 +6614,35 @@ NSC_DeriveKey(CK_SESSION_HANDLE hSession
3d912a
     }
3d912a
 
3d912a
     switch (mechanism) {
3d912a
+        /* get a public key from a private key. nsslowkey_ConvertToPublickey()
3d912a
+         * will generate the public portion if it doesn't already exist. */
3d912a
+        case CKM_NSS_PUB_FROM_PRIV: {
3d912a
+            NSSLOWKEYPrivateKey *privKey;
3d912a
+            NSSLOWKEYPublicKey *pubKey;
3d912a
+            int error;
3d912a
+
3d912a
+            crv = sftk_GetULongAttribute(sourceKey, CKA_KEY_TYPE, &keyType);
3d912a
+            if (crv != CKR_OK) {
3d912a
+                break;
3d912a
+            }
3d912a
+
3d912a
+            /* privKey is stored in sourceKey and will be destroyed when
3d912a
+             * the sourceKey is freed. */
3d912a
+            privKey = sftk_GetPrivKey(sourceKey, keyType, &crv;;
3d912a
+            if (privKey == NULL) {
3d912a
+                break;
3d912a
+            }
3d912a
+            pubKey = nsslowkey_ConvertToPublicKey(privKey);
3d912a
+            if (pubKey == NULL) {
3d912a
+                error = PORT_GetError();
3d912a
+                crv = sftk_MapCryptError(error);
3d912a
+                break;
3d912a
+            }
3d912a
+            crv = sftk_PutPubKey(key, sourceKey, keyType, pubKey);
3d912a
+            nsslowkey_DestroyPublicKey(pubKey);
3d912a
+            break;
3d912a
+        }
3d912a
+
3d912a
         case CKM_NSS_IKE_PRF_DERIVE:
3d912a
             if (pMechanism->ulParameterLen !=
3d912a
                 sizeof(CK_NSS_IKE_PRF_DERIVE_PARAMS)) {
3d912a
diff -up ./lib/softoken/pkcs11.c.pub-priv-mech ./lib/softoken/pkcs11.c
3d912a
--- ./lib/softoken/pkcs11.c.pub-priv-mech	2019-06-05 16:37:38.728685788 -0700
3d912a
+++ ./lib/softoken/pkcs11.c	2019-06-05 16:44:20.473479017 -0700
3d912a
@@ -2206,6 +2206,123 @@ sftk_GetPrivKey(SFTKObject *object, CK_K
3d912a
     return priv;
3d912a
 }
3d912a
 
3d912a
+/* populate a public key object from a lowpublic keys structure */
3d912a
+CK_RV
3d912a
+sftk_PutPubKey(SFTKObject *publicKey, SFTKObject *privateKey, CK_KEY_TYPE keyType, NSSLOWKEYPublicKey *pubKey)
3d912a
+{
3d912a
+    CK_OBJECT_CLASS classType = CKO_PUBLIC_KEY;
3d912a
+    CK_BBOOL cktrue = CK_TRUE;
3d912a
+    CK_RV crv = CKR_OK;
3d912a
+    sftk_DeleteAttributeType(publicKey, CKA_CLASS);
3d912a
+    sftk_DeleteAttributeType(publicKey, CKA_KEY_TYPE);
3d912a
+    sftk_DeleteAttributeType(publicKey, CKA_VALUE);
3d912a
+
3d912a
+    switch (keyType) {
3d912a
+        case CKK_RSA:
3d912a
+            sftk_DeleteAttributeType(publicKey, CKA_MODULUS);
3d912a
+            sftk_DeleteAttributeType(publicKey, CKA_PUBLIC_EXPONENT);
3d912a
+            /* format the keys */
3d912a
+            /* fill in the RSA dependent paramenters in the public key */
3d912a
+            crv = sftk_AddAttributeType(publicKey, CKA_MODULUS,
3d912a
+                                        sftk_item_expand(&pubKey->u.rsa.modulus));
3d912a
+            if (crv != CKR_OK)
3d912a
+                break;
3d912a
+            crv = sftk_AddAttributeType(publicKey, CKA_PUBLIC_EXPONENT,
3d912a
+                                        sftk_item_expand(&pubKey->u.rsa.publicExponent));
3d912a
+            break;
3d912a
+        case CKK_DSA:
3d912a
+            sftk_DeleteAttributeType(publicKey, CKA_PRIME);
3d912a
+            sftk_DeleteAttributeType(publicKey, CKA_SUBPRIME);
3d912a
+            sftk_DeleteAttributeType(publicKey, CKA_BASE);
3d912a
+            crv = sftk_AddAttributeType(publicKey, CKA_PRIME,
3d912a
+                                        sftk_item_expand(&pubKey->u.dsa.params.prime));
3d912a
+            if (crv != CKR_OK) {
3d912a
+                break;
3d912a
+            }
3d912a
+            crv = sftk_AddAttributeType(publicKey, CKA_SUBPRIME,
3d912a
+                                        sftk_item_expand(&pubKey->u.dsa.params.subPrime));
3d912a
+            if (crv != CKR_OK) {
3d912a
+                break;
3d912a
+            }
3d912a
+            crv = sftk_AddAttributeType(publicKey, CKA_BASE,
3d912a
+                                        sftk_item_expand(&pubKey->u.dsa.params.base));
3d912a
+            if (crv != CKR_OK) {
3d912a
+                break;
3d912a
+            }
3d912a
+            crv = sftk_AddAttributeType(publicKey, CKA_VALUE,
3d912a
+                                        sftk_item_expand(&pubKey->u.dsa.publicValue));
3d912a
+            break;
3d912a
+
3d912a
+        case CKK_DH:
3d912a
+            sftk_DeleteAttributeType(publicKey, CKA_PRIME);
3d912a
+            sftk_DeleteAttributeType(publicKey, CKA_BASE);
3d912a
+            crv = sftk_AddAttributeType(publicKey, CKA_PRIME,
3d912a
+                                        sftk_item_expand(&pubKey->u.dh.prime));
3d912a
+            if (crv != CKR_OK) {
3d912a
+                break;
3d912a
+            }
3d912a
+            crv = sftk_AddAttributeType(publicKey, CKA_BASE,
3d912a
+                                        sftk_item_expand(&pubKey->u.dh.base));
3d912a
+            if (crv != CKR_OK) {
3d912a
+                break;
3d912a
+            }
3d912a
+            crv = sftk_AddAttributeType(publicKey, CKA_VALUE,
3d912a
+                                        sftk_item_expand(&pubKey->u.dh.publicValue));
3d912a
+            break;
3d912a
+
3d912a
+        case CKK_EC:
3d912a
+            sftk_DeleteAttributeType(publicKey, CKA_EC_PARAMS);
3d912a
+            sftk_DeleteAttributeType(publicKey, CKA_EC_POINT);
3d912a
+
3d912a
+            crv = sftk_AddAttributeType(publicKey, CKA_EC_PARAMS,
3d912a
+                                        sftk_item_expand(&pubKey->u.ec.ecParams.DEREncoding));
3d912a
+            if (crv != CKR_OK) {
3d912a
+                break;
3d912a
+            }
3d912a
+
3d912a
+            crv = sftk_AddAttributeType(publicKey, CKA_EC_POINT,
3d912a
+                                        sftk_item_expand(&pubKey->u.ec.publicValue));
3d912a
+            break;
3d912a
+
3d912a
+        default:
3d912a
+            return CKR_KEY_TYPE_INCONSISTENT;
3d912a
+    }
3d912a
+    crv = sftk_AddAttributeType(publicKey, CKA_CLASS, &classType,
3d912a
+                                sizeof(CK_OBJECT_CLASS));
3d912a
+    if (crv != CKR_OK)
3d912a
+        return crv;
3d912a
+    crv = sftk_AddAttributeType(publicKey, CKA_KEY_TYPE, &keyType,
3d912a
+                                sizeof(CK_KEY_TYPE));
3d912a
+    if (crv != CKR_OK)
3d912a
+        return crv;
3d912a
+    /* now handle the operator attributes */
3d912a
+    if (sftk_isTrue(privateKey, CKA_DECRYPT)) {
3d912a
+        crv = sftk_forceAttribute(publicKey, CKA_ENCRYPT, &cktrue, sizeof(CK_BBOOL));
3d912a
+        if (crv != CKR_OK) {
3d912a
+            return crv;
3d912a
+        }
3d912a
+    }
3d912a
+    if (sftk_isTrue(privateKey, CKA_SIGN)) {
3d912a
+        crv = sftk_forceAttribute(publicKey, CKA_VERIFY, &cktrue, sizeof(CK_BBOOL));
3d912a
+        if (crv != CKR_OK) {
3d912a
+            return crv;
3d912a
+        }
3d912a
+    }
3d912a
+    if (sftk_isTrue(privateKey, CKA_SIGN_RECOVER)) {
3d912a
+        crv = sftk_forceAttribute(publicKey, CKA_VERIFY_RECOVER, &cktrue, sizeof(CK_BBOOL));
3d912a
+        if (crv != CKR_OK) {
3d912a
+            return crv;
3d912a
+        }
3d912a
+    }
3d912a
+    if (sftk_isTrue(privateKey, CKA_DERIVE)) {
3d912a
+        crv = sftk_forceAttribute(publicKey, CKA_DERIVE, &cktrue, sizeof(CK_BBOOL));
3d912a
+        if (crv != CKR_OK) {
3d912a
+            return crv;
3d912a
+        }
3d912a
+    }
3d912a
+    return crv;
3d912a
+}
3d912a
+
3d912a
 /*
3d912a
  **************************** Symetric Key utils ************************
3d912a
  */
3d912a
diff -up ./lib/softoken/pkcs11i.h.pub-priv-mech ./lib/softoken/pkcs11i.h
3d912a
--- ./lib/softoken/pkcs11i.h.pub-priv-mech	2019-06-05 16:37:38.730685787 -0700
3d912a
+++ ./lib/softoken/pkcs11i.h	2019-06-05 16:44:20.473479017 -0700
3d912a
@@ -695,6 +695,9 @@ extern NSSLOWKEYPublicKey *sftk_GetPubKe
3d912a
                                           CK_KEY_TYPE key_type, CK_RV *crvp);
3d912a
 extern NSSLOWKEYPrivateKey *sftk_GetPrivKey(SFTKObject *object,
3d912a
                                             CK_KEY_TYPE key_type, CK_RV *crvp);
3d912a
+extern CK_RV sftk_PutPubKey(SFTKObject *publicKey, SFTKObject *privKey,
3d912a
+                            CK_KEY_TYPE keyType,
3d912a
+                            NSSLOWKEYPublicKey *pubKey);
3d912a
 extern void sftk_FormatDESKey(unsigned char *key, int length);
3d912a
 extern PRBool sftk_CheckDESKey(unsigned char *key);
3d912a
 extern PRBool sftk_IsWeakKey(unsigned char *key, CK_KEY_TYPE key_type);
3d912a
diff -up ./lib/util/pkcs11n.h.pub-priv-mech ./lib/util/pkcs11n.h
3d912a
--- ./lib/util/pkcs11n.h.pub-priv-mech	2019-06-05 16:37:38.733685785 -0700
3d912a
+++ ./lib/util/pkcs11n.h	2019-06-05 16:44:54.389461561 -0700
3d912a
@@ -152,11 +152,6 @@
3d912a
 #define CKM_NSS_HKDF_SHA384 (CKM_NSS + 5)
3d912a
 #define CKM_NSS_HKDF_SHA512 (CKM_NSS + 6)
3d912a
 
3d912a
-/* IKE mechanism (to be proposed to PKCS #11 */
3d912a
-#define CKM_NSS_IKE_PRF_PLUS_DERIVE (CKM_NSS + 7)
3d912a
-#define CKM_NSS_IKE_PRF_DERIVE (CKM_NSS + 8)
3d912a
-#define CKM_NSS_IKE1_PRF_DERIVE (CKM_NSS + 9)
3d912a
-#define CKM_NSS_IKE1_APP_B_PRF_DERIVE (CKM_NSS + 10)
3d912a
 
3d912a
 /* J-PAKE round 1 key generation mechanisms.
3d912a
  *
3d912a
@@ -238,6 +233,15 @@
3d912a
 
3d912a
 #define CKM_NSS_CHACHA20_CTR (CKM_NSS + 33)
3d912a
 
3d912a
+/* IKE mechanism (to be proposed to PKCS #11 */
3d912a
+#define CKM_NSS_IKE_PRF_PLUS_DERIVE (CKM_NSS + 34)
3d912a
+#define CKM_NSS_IKE_PRF_DERIVE (CKM_NSS + 35)
3d912a
+#define CKM_NSS_IKE1_PRF_DERIVE (CKM_NSS + 36)
3d912a
+#define CKM_NSS_IKE1_APP_B_PRF_DERIVE (CKM_NSS + 37)
3d912a
+
3d912a
+/* Derive a public key from a bare private key */
3d912a
+#define CKM_NSS_PUB_FROM_PRIV (CKM_NSS + 40)
3d912a
+
3d912a
 /*
3d912a
  * HISTORICAL:
3d912a
  * Do not attempt to use these. They are only used by NETSCAPE's internal