Blame SOURCES/0026-EP11-Add-support-for-generating-and-importing-Kyber-.patch

447573
From 5b5d1830dadfbbd310c11d26d86426ed63eed936 Mon Sep 17 00:00:00 2001
447573
From: Ingo Franzki <ifranzki@linux.ibm.com>
447573
Date: Tue, 1 Mar 2022 11:09:26 +0100
447573
Subject: [PATCH 26/34] EP11: Add support for generating and importing Kyber
447573
 keys
447573
447573
Signed-off-by: Ingo Franzki <ifranzki@linux.ibm.com>
447573
---
447573
 usr/lib/ep11_stdll/ep11_specific.c | 222 ++++++++++++++++++++++---------------
447573
 1 file changed, 134 insertions(+), 88 deletions(-)
447573
447573
diff --git a/usr/lib/ep11_stdll/ep11_specific.c b/usr/lib/ep11_stdll/ep11_specific.c
447573
index 44796dba..bc17b07a 100644
447573
--- a/usr/lib/ep11_stdll/ep11_specific.c
447573
+++ b/usr/lib/ep11_stdll/ep11_specific.c
447573
@@ -3664,14 +3664,14 @@ import_DH_key_end:
447573
 }
447573
 
447573
 /*
447573
- * makes blobs for private imported IBM Dilithium keys and
447573
- * SPKIs for public imported IBM Dilithium keys.
447573
+ * makes blobs for private imported IBM PQC keys and
447573
+ * SPKIs for public imported IBM PQC keys.
447573
  * Similar to rawkey_2_blob, but keys must follow a standard BER encoding.
447573
  */
447573
-static CK_RV import_IBM_Dilithium_key(STDLL_TokData_t *tokdata, SESSION *sess,
447573
-                                      OBJECT *dilithium_key_obj,
447573
-                                      CK_BYTE *blob, size_t *blob_size,
447573
-                                      CK_BYTE *spki, size_t *spki_size)
447573
+static CK_RV import_IBM_pqc_key(STDLL_TokData_t *tokdata, SESSION *sess,
447573
+                                OBJECT *pqc_key_obj, CK_KEY_TYPE keytype,
447573
+                                CK_BYTE *blob, size_t *blob_size,
447573
+                                CK_BYTE *spki, size_t *spki_size)
447573
 {
447573
     ep11_private_data_t *ep11_data = tokdata->private_data;
447573
     CK_RV rc;
447573
@@ -3692,11 +3692,27 @@ static CK_RV import_IBM_Dilithium_key(STDLL_TokData_t *tokdata, SESSION *sess,
447573
     CK_ATTRIBUTE *value_attr = NULL;
447573
     CK_BBOOL data_alloced = TRUE;
447573
     const struct pqc_oid *oid;
447573
+    const char *key_type_str;
447573
+    CK_MECHANISM_TYPE pqc_mech;
447573
+
447573
+    switch (keytype) {
447573
+    case CKK_IBM_PQC_DILITHIUM:
447573
+        key_type_str = "Dilithium";
447573
+        pqc_mech = CKM_IBM_DILITHIUM;
447573
+        break;
447573
+    case CKK_IBM_PQC_KYBER:
447573
+        key_type_str = "Kyber";
447573
+        pqc_mech = CKM_IBM_KYBER;
447573
+        break;
447573
+    default:
447573
+        TRACE_ERROR("Invalid key type provided for %s\n ", __func__);
447573
+        return CKR_KEY_TYPE_INCONSISTENT;
447573
+    }
447573
 
447573
     memcpy(iv, "1234567812345678", AES_BLOCK_SIZE);
447573
 
447573
     /* need class for secret/public key info */
447573
-    rc = template_attribute_get_ulong(dilithium_key_obj->template, CKA_CLASS,
447573
+    rc = template_attribute_get_ulong(pqc_key_obj->template, CKA_CLASS,
447573
                                       &class);
447573
     if (rc != CKR_OK) {
447573
         TRACE_ERROR("Could not find CKA_CLASS for the key.\n");
447573
@@ -3706,20 +3722,20 @@ static CK_RV import_IBM_Dilithium_key(STDLL_TokData_t *tokdata, SESSION *sess,
447573
     /* m_Unwrap builds key blob in the card,
447573
      * tell ep11 the attributes the user specified for that key.
447573
      */
447573
-    rc = build_ep11_attrs(tokdata, dilithium_key_obj->template,
447573
+    rc = build_ep11_attrs(tokdata, pqc_key_obj->template,
447573
                           &p_attrs, &attrs_len,
447573
-                          CKK_IBM_PQC_DILITHIUM, class, -1, &mech_w);
447573
+                          keytype, class, -1, &mech_w);
447573
     if (rc != CKR_OK)
447573
         goto done;
447573
 
447573
     if (class != CKO_PRIVATE_KEY) {
447573
-        /* Make an SPKI for the public IBM Dilithium key */
447573
+        /* Make an SPKI for the public IBM PQC key */
447573
 
447573
-        /* A public IBM Dilithium key must either have a CKA_VALUE containing
447573
+        /* A public IBM PQC key must either have a CKA_VALUE containing
447573
          * the SPKI, or must have a keyform/mode value and the individual
447573
          * attributes
447573
          */
447573
-        if (template_attribute_find(dilithium_key_obj->template,
447573
+        if (template_attribute_find(pqc_key_obj->template,
447573
                                     CKA_VALUE, &value_attr) &&
447573
             value_attr->ulValueLen > 0 && value_attr ->pValue != NULL) {
447573
             /* CKA_VALUE with SPKI */
447573
@@ -3731,16 +3747,16 @@ static CK_RV import_IBM_Dilithium_key(STDLL_TokData_t *tokdata, SESSION *sess,
447573
              * Decode SPKI and add public key attributes. This also adds the
447573
              * keyform and mode attributes to the template.
447573
              */
447573
-            rc = ibm_dilithium_priv_unwrap_get_data(dilithium_key_obj->template,
447573
-                                                    data, data_len, FALSE);
447573
+            rc = ibm_pqc_priv_unwrap_get_data(pqc_key_obj->template, keytype,
447573
+                                              data, data_len, FALSE);
447573
             if (rc != CKR_OK) {
447573
                 TRACE_ERROR("Failed to decode SPKI from CKA_VALUE.\n");
447573
                 goto done;
447573
             }
447573
          } else {
447573
             /* Individual attributes */
447573
-             rc = ibm_dilithium_publ_get_spki(dilithium_key_obj->template,
447573
-                                              FALSE, &data, &data_len);
447573
+             rc = ibm_pqc_publ_get_spki(pqc_key_obj->template, keytype,
447573
+                                        FALSE, &data, &data_len);
447573
             if (rc != CKR_OK) {
447573
                 TRACE_ERROR("%s public key import class=0x%lx rc=0x%lx "
447573
                             "data_len=0x%lx\n", __func__, class, rc, data_len);
447573
@@ -3751,15 +3767,13 @@ static CK_RV import_IBM_Dilithium_key(STDLL_TokData_t *tokdata, SESSION *sess,
447573
             }
447573
 
447573
             /* Ensure both, keyform and mode attributes are added */
447573
-            oid = ibm_pqc_get_keyform_mode(dilithium_key_obj->template,
447573
-                                           CKM_IBM_DILITHIUM);
447573
+            oid = ibm_pqc_get_keyform_mode(pqc_key_obj->template, pqc_mech);
447573
             if (oid == NULL) {
447573
                 rc = CKR_TEMPLATE_INCOMPLETE;
447573
                 goto done;
447573
             }
447573
 
447573
-            rc = ibm_pqc_add_keyform_mode(dilithium_key_obj->template,
447573
-                                          oid, CKM_IBM_DILITHIUM);
447573
+            rc = ibm_pqc_add_keyform_mode(pqc_key_obj->template, oid, pqc_mech);
447573
             if (rc != CKR_OK) {
447573
                 TRACE_ERROR("ibm_pqc_add_keyform_mode failed\n");
447573
                 goto done;
447573
@@ -3772,7 +3786,7 @@ static CK_RV import_IBM_Dilithium_key(STDLL_TokData_t *tokdata, SESSION *sess,
447573
                 goto done;
447573
             }
447573
 
447573
-            rc = template_update_attribute(dilithium_key_obj->template,
447573
+            rc = template_update_attribute(pqc_key_obj->template,
447573
                                            value_attr);
447573
             if (rc != CKR_OK) {
447573
                 TRACE_ERROR("%s template_update_attribute failed with rc=0x%lx\n",
447573
@@ -3786,7 +3800,7 @@ static CK_RV import_IBM_Dilithium_key(STDLL_TokData_t *tokdata, SESSION *sess,
447573
         /* save the SPKI as blob although it is not a blob.
447573
          * The card expects MACed-SPKIs as public keys.
447573
          */
447573
-        rc = make_maced_spki(tokdata, sess, dilithium_key_obj, data, data_len,
447573
+        rc = make_maced_spki(tokdata, sess, pqc_key_obj, data, data_len,
447573
                              blob, blob_size, -1);
447573
         if (rc != CKR_OK) {
447573
             TRACE_ERROR("%s failed to make a MACed-SPKI rc=0x%lx\n",
447573
@@ -3798,13 +3812,13 @@ static CK_RV import_IBM_Dilithium_key(STDLL_TokData_t *tokdata, SESSION *sess,
447573
 
447573
     } else {
447573
 
447573
-        /* imported private IBM Dilithium key goes here */
447573
+        /* imported private IBM PQC key goes here */
447573
 
447573
-        /* A public IBM Dilithium key must either have a CKA_VALUE containing
447573
+        /* A public IBM PQC key must either have a CKA_VALUE containing
447573
          * the PKCS#8 encoded private key, or must have a keyform/mode value
447573
          * and the individual attributes
447573
          */
447573
-        if (template_attribute_find(dilithium_key_obj->template,
447573
+        if (template_attribute_find(pqc_key_obj->template,
447573
                                     CKA_VALUE, &value_attr) &&
447573
             value_attr->ulValueLen > 0 && value_attr ->pValue != NULL) {
447573
             /* CKA_VALUE with SPKI */
447573
@@ -3813,8 +3827,8 @@ static CK_RV import_IBM_Dilithium_key(STDLL_TokData_t *tokdata, SESSION *sess,
447573
             data_alloced = FALSE;
447573
 
447573
             /* Decode PKCS#8 private key and add key attributes */
447573
-            rc = ibm_dilithium_priv_unwrap(dilithium_key_obj->template,
447573
-                                           data, data_len, FALSE);
447573
+            rc = ibm_pqc_priv_unwrap(pqc_key_obj->template, keytype,
447573
+                                     data, data_len, FALSE);
447573
             if (rc != CKR_OK) {
447573
                 TRACE_ERROR("Failed to decode private key from CKA_VALUE.\n");
447573
                 goto done;
447573
@@ -3824,23 +3838,22 @@ static CK_RV import_IBM_Dilithium_key(STDLL_TokData_t *tokdata, SESSION *sess,
447573
              * padding is done in mechanism. This also adds the keyform and mode
447573
              * attributes to the template.
447573
              */
447573
-            rc = ibm_dilithium_priv_wrap_get_data(dilithium_key_obj->template,
447573
-                                                  FALSE, &data, &data_len);
447573
+            rc = ibm_pqc_priv_wrap_get_data(pqc_key_obj->template, keytype,
447573
+                                            FALSE, &data, &data_len);
447573
             if (rc != CKR_OK) {
447573
-                TRACE_DEVEL("%s Dilithium wrap get data failed\n", __func__);
447573
+                TRACE_DEVEL("%s %s wrap get data failed\n", __func__,
447573
+                            key_type_str);
447573
                 goto done;
447573
             }
447573
 
447573
             /* Ensure both, keyform and mode attributes are added */
447573
-            oid = ibm_pqc_get_keyform_mode(dilithium_key_obj->template,
447573
-                                           CKM_IBM_DILITHIUM);
447573
+            oid = ibm_pqc_get_keyform_mode(pqc_key_obj->template, pqc_mech);
447573
             if (oid == NULL) {
447573
                 rc = CKR_TEMPLATE_INCOMPLETE;
447573
                 goto done;
447573
             }
447573
 
447573
-            rc = ibm_pqc_add_keyform_mode(dilithium_key_obj->template,
447573
-                                          oid, CKM_IBM_DILITHIUM);
447573
+            rc = ibm_pqc_add_keyform_mode(pqc_key_obj->template, oid, pqc_mech);
447573
             if (rc != CKR_OK) {
447573
                 TRACE_ERROR("ibm_pqc_add_keyform_mode failed\n");
447573
                 goto done;
447573
@@ -3849,8 +3862,8 @@ static CK_RV import_IBM_Dilithium_key(STDLL_TokData_t *tokdata, SESSION *sess,
447573
 
447573
         /* encrypt */
447573
         RETRY_START(rc, tokdata)
447573
-            if (ep11_pqc_obj_strength_supported(target_info, CKM_IBM_DILITHIUM,
447573
-                                                dilithium_key_obj))
447573
+            if (ep11_pqc_obj_strength_supported(target_info, pqc_mech,
447573
+                                                pqc_key_obj))
447573
                 rc = dll_m_EncryptSingle(ep11_data->raw2key_wrap_blob,
447573
                                          ep11_data->raw2key_wrap_blob_l,
447573
                                          &mech_w, data, data_len,
447573
@@ -3870,8 +3883,7 @@ static CK_RV import_IBM_Dilithium_key(STDLL_TokData_t *tokdata, SESSION *sess,
447573
             goto done;
447573
         }
447573
 
447573
-        rc = check_key_attributes(tokdata, CKK_IBM_PQC_DILITHIUM,
447573
-                            CKO_PRIVATE_KEY,
447573
+        rc = check_key_attributes(tokdata, keytype, CKO_PRIVATE_KEY,
447573
                             p_attrs, attrs_len,
447573
                             &new_p_attrs, &new_attrs_len, -1);
447573
         if (rc != CKR_OK) {
447573
@@ -3880,12 +3892,12 @@ static CK_RV import_IBM_Dilithium_key(STDLL_TokData_t *tokdata, SESSION *sess,
447573
             goto done;
447573
         }
447573
 
447573
-        trace_attributes(__func__, "Dilithium import:", new_p_attrs, new_attrs_len);
447573
+        trace_attributes(__func__, "PQC import:", new_p_attrs, new_attrs_len);
447573
 
447573
-        ep11_get_pin_blob(ep11_session, object_is_session_object(dilithium_key_obj),
447573
+        ep11_get_pin_blob(ep11_session, object_is_session_object(pqc_key_obj),
447573
                           &ep11_pin_blob, &ep11_pin_blob_len);
447573
 
447573
-        /* calls the card, it decrypts the private Dilithium key,
447573
+        /* calls the card, it decrypts the private PQC key,
447573
          * reads its BER format and builds a blob.
447573
          */
447573
         RETRY_START(rc, tokdata)
447573
@@ -3908,12 +3920,20 @@ static CK_RV import_IBM_Dilithium_key(STDLL_TokData_t *tokdata, SESSION *sess,
447573
                        __func__, rc, *blob_size);
447573
         }
447573
 
447573
-        cleanse_attribute(dilithium_key_obj->template, CKA_VALUE);
447573
-        cleanse_attribute(dilithium_key_obj->template, CKA_IBM_DILITHIUM_SEED);
447573
-        cleanse_attribute(dilithium_key_obj->template, CKA_IBM_DILITHIUM_TR);
447573
-        cleanse_attribute(dilithium_key_obj->template, CKA_IBM_DILITHIUM_S1);
447573
-        cleanse_attribute(dilithium_key_obj->template, CKA_IBM_DILITHIUM_S2);
447573
-        cleanse_attribute(dilithium_key_obj->template, CKA_IBM_DILITHIUM_T0);
447573
+        cleanse_attribute(pqc_key_obj->template, CKA_VALUE);
447573
+
447573
+        switch (keytype) {
447573
+        case CKK_IBM_PQC_DILITHIUM:
447573
+            cleanse_attribute(pqc_key_obj->template, CKA_IBM_DILITHIUM_SEED);
447573
+            cleanse_attribute(pqc_key_obj->template, CKA_IBM_DILITHIUM_TR);
447573
+            cleanse_attribute(pqc_key_obj->template, CKA_IBM_DILITHIUM_S1);
447573
+            cleanse_attribute(pqc_key_obj->template, CKA_IBM_DILITHIUM_S2);
447573
+            cleanse_attribute(pqc_key_obj->template, CKA_IBM_DILITHIUM_T0);
447573
+            break;
447573
+        case CKK_IBM_PQC_KYBER:
447573
+            cleanse_attribute(pqc_key_obj->template, CKA_IBM_KYBER_SK);
447573
+            break;
447573
+        }
447573
     }
447573
 
447573
 done:
447573
@@ -4020,15 +4040,16 @@ CK_RV token_specific_object_add(STDLL_TokData_t * tokdata, SESSION * sess,
447573
                    __func__, rc, blobsize);
447573
         break;
447573
     case CKK_IBM_PQC_DILITHIUM:
447573
-        rc = import_IBM_Dilithium_key(tokdata, sess, obj, blob, &blobsize,
447573
-                                      spki, &spkisize);
447573
+    case CKK_IBM_PQC_KYBER:
447573
+        rc = import_IBM_pqc_key(tokdata, sess, obj, keytype, blob, &blobsize,
447573
+                                spki, &spkisize);
447573
         if (rc != CKR_OK) {
447573
-            TRACE_ERROR("%s import IBM Dilithium key rc=0x%lx blobsize=0x%zx\n",
447573
-                        __func__, rc, blobsize);
447573
+            TRACE_ERROR("%s import IBM PQC key kytype=0x%lx rc=0x%lx blobsize=0x%zx\n",
447573
+                        __func__, keytype, rc, blobsize);
447573
             return rc;
447573
         }
447573
-        TRACE_INFO("%s import IBM Dilithium key rc=0x%lx blobsize=0x%zx\n",
447573
-                   __func__, rc, blobsize);
447573
+        TRACE_INFO("%s import IBM PQC key kytype=0x%lx rc=0x%lx blobsize=0x%zx\n",
447573
+                   __func__, keytype, rc, blobsize);
447573
         break;
447573
     case CKK_DES2:
447573
     case CKK_DES3:
447573
@@ -6582,10 +6603,10 @@ error:
447573
     return rc;
447573
 }
447573
 
447573
-static CK_RV ibm_dilithium_generate_keypair(STDLL_TokData_t *tokdata,
447573
-                                     SESSION *sess,
447573
-                                     CK_MECHANISM_PTR pMechanism,
447573
-                                     TEMPLATE *publ_tmpl, TEMPLATE *priv_tmpl)
447573
+static CK_RV ibm_pqc_generate_keypair(STDLL_TokData_t *tokdata,
447573
+                                      SESSION *sess,
447573
+                                      CK_MECHANISM_PTR pMechanism,
447573
+                                      TEMPLATE *publ_tmpl, TEMPLATE *priv_tmpl)
447573
 {
447573
     CK_RV rc;
447573
     CK_ATTRIBUTE *attr = NULL;
447573
@@ -6593,7 +6614,7 @@ static CK_RV ibm_dilithium_generate_keypair(STDLL_TokData_t *tokdata,
447573
     size_t privkey_blob_len = sizeof(privkey_blob);
447573
     unsigned char spki[MAX_BLOBSIZE];
447573
     size_t spki_len = sizeof(spki);
447573
-    CK_ULONG ktype = CKK_IBM_PQC_DILITHIUM;
447573
+    CK_ULONG ktype;
447573
     unsigned char *ep11_pin_blob = NULL;
447573
     CK_ULONG ep11_pin_blob_len = 0;
447573
     ep11_session_t *ep11_session = (ep11_session_t *) sess->private_data;
447573
@@ -6601,9 +6622,19 @@ static CK_RV ibm_dilithium_generate_keypair(STDLL_TokData_t *tokdata,
447573
     CK_ULONG new_publ_attrs_len = 0, new_priv_attrs_len = 0;
447573
     CK_ATTRIBUTE *new_publ_attrs2 = NULL, *new_priv_attrs2 = NULL;
447573
     CK_ULONG new_publ_attrs2_len = 0, new_priv_attrs2_len = 0;
447573
-    const struct pqc_oid *dilithium_oid;
447573
+    const struct pqc_oid *pqc_oid;
447573
+    const char *key_type_str;
447573
 
447573
-    if (pMechanism->mechanism != CKM_IBM_DILITHIUM) {
447573
+    switch (pMechanism->mechanism) {
447573
+    case CKM_IBM_DILITHIUM:
447573
+        key_type_str = "Dilithium";
447573
+        ktype = CKK_IBM_PQC_DILITHIUM;
447573
+        break;
447573
+    case CKM_IBM_KYBER:
447573
+        key_type_str = "Kyber";
447573
+        ktype = CKK_IBM_PQC_KYBER;
447573
+        break;
447573
+    default:
447573
         TRACE_ERROR("Invalid mechanism provided for %s\n ", __func__);
447573
         return CKR_MECHANISM_INVALID;
447573
     }
447573
@@ -6624,25 +6655,37 @@ static CK_RV ibm_dilithium_generate_keypair(STDLL_TokData_t *tokdata,
447573
         goto error;
447573
     }
447573
 
447573
-    dilithium_oid = ibm_pqc_get_keyform_mode(publ_tmpl, CKM_IBM_DILITHIUM);
447573
-    if (dilithium_oid == NULL)
447573
-        dilithium_oid = ibm_pqc_get_keyform_mode(priv_tmpl, CKM_IBM_DILITHIUM);
447573
-    if (dilithium_oid == NULL)
447573
-        dilithium_oid = find_pqc_by_keyform(dilithium_oids,
447573
-                                            CK_IBM_DILITHIUM_KEYFORM_ROUND2_65);
447573
-    if (dilithium_oid == NULL) {
447573
-        TRACE_ERROR("%s Failed to determine Dilithium OID\n", __func__);
447573
+    pqc_oid = ibm_pqc_get_keyform_mode(publ_tmpl, pMechanism->mechanism);
447573
+    if (pqc_oid == NULL)
447573
+        pqc_oid = ibm_pqc_get_keyform_mode(priv_tmpl, pMechanism->mechanism);
447573
+    if (pqc_oid == NULL) {
447573
+        switch (pMechanism->mechanism) {
447573
+        case CKM_IBM_DILITHIUM:
447573
+            pqc_oid = find_pqc_by_keyform(dilithium_oids,
447573
+                                          CK_IBM_DILITHIUM_KEYFORM_ROUND2_65);
447573
+            break;
447573
+        case CKM_IBM_KYBER:
447573
+            pqc_oid = find_pqc_by_keyform(kyber_oids,
447573
+                                          CK_IBM_KYBER_KEYFORM_ROUND2_1024);
447573
+            break;
447573
+        default:
447573
+            /* pqc_oid stays NULL */
447573
+            break;
447573
+        }
447573
+    }
447573
+    if (pqc_oid == NULL) {
447573
+        TRACE_ERROR("%s Failed to determine %s OID\n", __func__, key_type_str);
447573
         rc = CKR_FUNCTION_FAILED;
447573
         goto error;
447573
     }
447573
 
447573
-    TRACE_INFO("%s Generate Dilithium key with keyform %lu\n", __func__,
447573
-               dilithium_oid->keyform);
447573
+    TRACE_INFO("%s Generate %s key with keyform %lu\n", __func__, key_type_str,
447573
+               pqc_oid->keyform);
447573
 
447573
     rc = add_to_attribute_array(&new_publ_attrs, &new_publ_attrs_len,
447573
                                 CKA_IBM_PQC_PARAMS,
447573
-                                (CK_BYTE *)dilithium_oid->oid,
447573
-                                dilithium_oid->oid_len);
447573
+                                (CK_BYTE *)pqc_oid->oid,
447573
+                                pqc_oid->oid_len);
447573
     if (rc != CKR_OK) {
447573
         TRACE_ERROR("%s add_to_attribute_array failed with rc=0x%lx\n",
447573
                     __func__, rc);
447573
@@ -6651,8 +6694,8 @@ static CK_RV ibm_dilithium_generate_keypair(STDLL_TokData_t *tokdata,
447573
 
447573
     rc = add_to_attribute_array(&new_priv_attrs, &new_priv_attrs_len,
447573
                                 CKA_IBM_PQC_PARAMS,
447573
-                                (CK_BYTE *)dilithium_oid->oid,
447573
-                                dilithium_oid->oid_len);
447573
+                                (CK_BYTE *)pqc_oid->oid,
447573
+                                pqc_oid->oid_len);
447573
     if (rc != CKR_OK) {
447573
         TRACE_ERROR("%s add_to_attribute_array failed with rc=0x%lx\n",
447573
                     __func__, rc);
447573
@@ -6663,8 +6706,8 @@ static CK_RV ibm_dilithium_generate_keypair(STDLL_TokData_t *tokdata,
447573
                               new_publ_attrs, new_publ_attrs_len,
447573
                               &new_publ_attrs2, &new_publ_attrs2_len, -1);
447573
     if (rc != CKR_OK) {
447573
-        TRACE_ERROR("%s Dilithium check public key attributes failed with "
447573
-                    "rc=0x%lx\n", __func__, rc);
447573
+        TRACE_ERROR("%s %s check public key attributes failed with "
447573
+                    "rc=0x%lx\n", __func__, key_type_str, rc);
447573
         goto error;
447573
     }
447573
 
447573
@@ -6672,14 +6715,14 @@ static CK_RV ibm_dilithium_generate_keypair(STDLL_TokData_t *tokdata,
447573
                               new_priv_attrs, new_priv_attrs_len,
447573
                               &new_priv_attrs2, &new_priv_attrs2_len, -1);
447573
     if (rc != CKR_OK) {
447573
-        TRACE_ERROR("%s Dilithium check private key attributes failed with "
447573
-                    "rc=0x%lx\n", __func__, rc);
447573
+        TRACE_ERROR("%s %s check private key attributes failed with "
447573
+                    "rc=0x%lx\n", __func__, key_type_str, rc);
447573
         goto error;
447573
     }
447573
 
447573
-    trace_attributes(__func__, "Dilithium public key attributes:",
447573
+    trace_attributes(__func__, "PQC public key attributes:",
447573
                      new_publ_attrs2, new_publ_attrs2_len);
447573
-    trace_attributes(__func__, "Dilithium private key attributes:",
447573
+    trace_attributes(__func__, "PQC private key attributes:",
447573
                      new_priv_attrs2, new_priv_attrs2_len);
447573
 
447573
     ep11_get_pin_blob(ep11_session,
447573
@@ -6691,7 +6734,7 @@ static CK_RV ibm_dilithium_generate_keypair(STDLL_TokData_t *tokdata,
447573
 
447573
     RETRY_START(rc, tokdata)
447573
         if (ep11_pqc_strength_supported(target_info, pMechanism->mechanism,
447573
-                                        dilithium_oid))
447573
+                                        pqc_oid))
447573
             rc = dll_m_GenerateKeyPair(pMechanism,
447573
                                        new_publ_attrs2, new_publ_attrs2_len,
447573
                                        new_priv_attrs2, new_priv_attrs2_len,
447573
@@ -6752,16 +6795,18 @@ static CK_RV ibm_dilithium_generate_keypair(STDLL_TokData_t *tokdata,
447573
         goto error;
447573
     }
447573
 
447573
-    rc = ibm_dilithium_priv_unwrap_get_data(publ_tmpl, spki, spki_len, TRUE);
447573
+    rc = ibm_pqc_priv_unwrap_get_data(publ_tmpl, ktype,
447573
+                                      spki, spki_len, TRUE);
447573
     if (rc != CKR_OK) {
447573
-        TRACE_ERROR("%s ibm_dilithium_priv_unwrap_get_data with rc=0x%lx\n",
447573
+        TRACE_ERROR("%s ibm_pqc_priv_unwrap_get_data with rc=0x%lx\n",
447573
                     __func__, rc);
447573
         goto error;
447573
     }
447573
 
447573
-    rc = ibm_dilithium_priv_unwrap_get_data(priv_tmpl, spki, spki_len, FALSE);
447573
+    rc = ibm_pqc_priv_unwrap_get_data(priv_tmpl, ktype,
447573
+                                      spki, spki_len, FALSE);
447573
     if (rc != CKR_OK) {
447573
-        TRACE_ERROR("%s ibm_dilithium_priv_unwrap_get_data with rc=0x%lx\n",
447573
+        TRACE_ERROR("%s ibm_pqc_priv_unwrap_get_data with rc=0x%lx\n",
447573
                     __func__, rc);
447573
         goto error;
447573
     }
447573
@@ -6854,9 +6899,10 @@ CK_RV ep11tok_generate_key_pair(STDLL_TokData_t * tokdata, SESSION * sess,
447573
                                   private_key_obj->template);
447573
         break;
447573
     case CKM_IBM_DILITHIUM:
447573
-        rc = ibm_dilithium_generate_keypair(tokdata, sess, pMechanism,
447573
-                                            public_key_obj->template,
447573
-                                            private_key_obj->template);
447573
+    case CKM_IBM_KYBER:
447573
+        rc = ibm_pqc_generate_keypair(tokdata, sess, pMechanism,
447573
+                                      public_key_obj->template,
447573
+                                      private_key_obj->template);
447573
         break;
447573
     default:
447573
         TRACE_ERROR("%s invalid mech %s\n", __func__,
447573
-- 
447573
2.16.2.windows.1
447573