|
 |
971d89 |
From ee3464dff7536c98fd64c80d87c765f703dff0f0 Mon Sep 17 00:00:00 2001
|
|
|
971d89 |
From: Ingo Franzki <ifranzki@linux.ibm.com>
|
|
|
971d89 |
Date: Fri, 18 Nov 2022 10:44:56 +0100
|
|
|
971d89 |
Subject: [PATCH 11/34] EP11: remove dead code and unused variables
|
|
|
971d89 |
|
|
|
971d89 |
Some variables are declared and initialized to NULL, but never used or set.
|
|
|
971d89 |
The attempt to free them at the end of the function is dead code.
|
|
|
971d89 |
|
|
|
971d89 |
Signed-off-by: Ingo Franzki <ifranzki@linux.ibm.com>
|
|
|
971d89 |
---
|
|
|
971d89 |
usr/lib/ep11_stdll/ep11_specific.c | 28 ----------------------------
|
|
|
971d89 |
1 file changed, 28 deletions(-)
|
|
|
971d89 |
|
|
|
971d89 |
diff --git a/usr/lib/ep11_stdll/ep11_specific.c b/usr/lib/ep11_stdll/ep11_specific.c
|
|
|
971d89 |
index 9e320503..1dbfe0f5 100644
|
|
|
971d89 |
--- a/usr/lib/ep11_stdll/ep11_specific.c
|
|
|
971d89 |
+++ b/usr/lib/ep11_stdll/ep11_specific.c
|
|
|
971d89 |
@@ -3575,7 +3575,6 @@ static CK_RV import_IBM_Dilithium_key(STDLL_TokData_t *tokdata, SESSION *sess,
|
|
|
971d89 |
unsigned char *ep11_pin_blob = NULL;
|
|
|
971d89 |
CK_ULONG ep11_pin_blob_len = 0;
|
|
|
971d89 |
ep11_session_t *ep11_session = (ep11_session_t *) sess->private_data;
|
|
|
971d89 |
- CK_BYTE *pubkey = NULL;
|
|
|
971d89 |
|
|
|
971d89 |
memcpy(iv, "1234567812345678", AES_BLOCK_SIZE);
|
|
|
971d89 |
|
|
|
971d89 |
@@ -3734,9 +3733,6 @@ static CK_RV import_IBM_Dilithium_key(STDLL_TokData_t *tokdata, SESSION *sess,
|
|
|
971d89 |
}
|
|
|
971d89 |
|
|
|
971d89 |
done:
|
|
|
971d89 |
-
|
|
|
971d89 |
- if (pubkey)
|
|
|
971d89 |
- free(pubkey);
|
|
|
971d89 |
if (data) {
|
|
|
971d89 |
OPENSSL_cleanse(data, data_len);
|
|
|
971d89 |
free(data);
|
|
|
971d89 |
@@ -5486,7 +5482,6 @@ static CK_RV dh_generate_keypair(STDLL_TokData_t *tokdata,
|
|
|
971d89 |
CK_ATTRIBUTE *opaque_attr = NULL;
|
|
|
971d89 |
CK_ATTRIBUTE *value_attr = NULL;
|
|
|
971d89 |
CK_ATTRIBUTE *attr = NULL;
|
|
|
971d89 |
- CK_ATTRIBUTE *pPublicKeyTemplate_new = NULL;
|
|
|
971d89 |
CK_ATTRIBUTE_PTR dh_pPublicKeyTemplate = NULL;
|
|
|
971d89 |
CK_ULONG dh_ulPublicKeyAttributeCount = 0;
|
|
|
971d89 |
CK_ATTRIBUTE_PTR dh_pPrivateKeyTemplate = NULL;
|
|
|
971d89 |
@@ -5746,7 +5741,6 @@ static CK_RV dh_generate_keypair(STDLL_TokData_t *tokdata,
|
|
|
971d89 |
}
|
|
|
971d89 |
|
|
|
971d89 |
dh_generate_keypair_end:
|
|
|
971d89 |
- free(pPublicKeyTemplate_new);
|
|
|
971d89 |
if (dh_pgs.pg != NULL)
|
|
|
971d89 |
free(dh_pgs.pg);
|
|
|
971d89 |
if (dh_pPublicKeyTemplate)
|
|
|
971d89 |
@@ -5778,7 +5772,6 @@ static CK_RV dsa_generate_keypair(STDLL_TokData_t *tokdata,
|
|
|
971d89 |
CK_ATTRIBUTE *opaque_attr = NULL;
|
|
|
971d89 |
CK_ATTRIBUTE *value_attr = NULL;
|
|
|
971d89 |
CK_ATTRIBUTE *attr = NULL;
|
|
|
971d89 |
- CK_ATTRIBUTE *pPublicKeyTemplate_new = NULL;
|
|
|
971d89 |
CK_BYTE *key;
|
|
|
971d89 |
CK_BYTE *data, *oid, *parm;
|
|
|
971d89 |
CK_ULONG data_len, field_len, bit_str_len, oid_len, parm_len;
|
|
|
971d89 |
@@ -6047,7 +6040,6 @@ static CK_RV dsa_generate_keypair(STDLL_TokData_t *tokdata,
|
|
|
971d89 |
}
|
|
|
971d89 |
|
|
|
971d89 |
dsa_generate_keypair_end:
|
|
|
971d89 |
- free(pPublicKeyTemplate_new);
|
|
|
971d89 |
if (dsa_pqgs.pqg != NULL)
|
|
|
971d89 |
free(dsa_pqgs.pqg);
|
|
|
971d89 |
if (dsa_pPublicKeyTemplate)
|
|
|
971d89 |
@@ -6080,10 +6072,6 @@ static CK_RV rsa_ec_generate_keypair(STDLL_TokData_t *tokdata,
|
|
|
971d89 |
CK_BYTE *data, *oid, *parm;
|
|
|
971d89 |
CK_ULONG data_len, oid_len, parm_len;
|
|
|
971d89 |
CK_ULONG field_len;
|
|
|
971d89 |
- CK_ATTRIBUTE_PTR new_pPublicKeyTemplate = NULL;
|
|
|
971d89 |
- CK_ULONG new_ulPublicKeyAttributeCount = 0;
|
|
|
971d89 |
- CK_ATTRIBUTE_PTR new_pPrivateKeyTemplate = NULL;
|
|
|
971d89 |
- CK_ULONG new_ulPrivateKeyAttributeCount = 0;
|
|
|
971d89 |
CK_ULONG ktype;
|
|
|
971d89 |
unsigned char *ep11_pin_blob = NULL;
|
|
|
971d89 |
CK_ULONG ep11_pin_blob_len = 0;
|
|
|
971d89 |
@@ -6399,12 +6387,6 @@ static CK_RV rsa_ec_generate_keypair(STDLL_TokData_t *tokdata,
|
|
|
971d89 |
}
|
|
|
971d89 |
|
|
|
971d89 |
error:
|
|
|
971d89 |
- if (new_pPrivateKeyTemplate)
|
|
|
971d89 |
- free_attribute_array(new_pPrivateKeyTemplate,
|
|
|
971d89 |
- new_ulPrivateKeyAttributeCount);
|
|
|
971d89 |
- if (new_pPublicKeyTemplate)
|
|
|
971d89 |
- free_attribute_array(new_pPublicKeyTemplate,
|
|
|
971d89 |
- new_ulPublicKeyAttributeCount);
|
|
|
971d89 |
if (new_publ_attrs)
|
|
|
971d89 |
free_attribute_array(new_publ_attrs, new_publ_attrs_len);
|
|
|
971d89 |
if (new_priv_attrs)
|
|
|
971d89 |
@@ -6432,10 +6414,6 @@ static CK_RV ibm_dilithium_generate_keypair(STDLL_TokData_t *tokdata,
|
|
|
971d89 |
CK_BYTE *data, *oid, *parm;
|
|
|
971d89 |
CK_ULONG data_len, oid_len, parm_len;
|
|
|
971d89 |
CK_ULONG field_len;
|
|
|
971d89 |
- CK_ATTRIBUTE_PTR new_pPublicKeyTemplate = NULL;
|
|
|
971d89 |
- CK_ULONG new_ulPublicKeyAttributeCount = 0;
|
|
|
971d89 |
- CK_ATTRIBUTE_PTR new_pPrivateKeyTemplate = NULL;
|
|
|
971d89 |
- CK_ULONG new_ulPrivateKeyAttributeCount = 0;
|
|
|
971d89 |
CK_ULONG ktype = CKK_IBM_PQC_DILITHIUM;
|
|
|
971d89 |
unsigned char *ep11_pin_blob = NULL;
|
|
|
971d89 |
CK_ULONG ep11_pin_blob_len = 0;
|
|
|
971d89 |
@@ -6679,12 +6657,6 @@ static CK_RV ibm_dilithium_generate_keypair(STDLL_TokData_t *tokdata,
|
|
|
971d89 |
}
|
|
|
971d89 |
|
|
|
971d89 |
error:
|
|
|
971d89 |
- if (new_pPrivateKeyTemplate)
|
|
|
971d89 |
- free_attribute_array(new_pPrivateKeyTemplate,
|
|
|
971d89 |
- new_ulPrivateKeyAttributeCount);
|
|
|
971d89 |
- if (new_pPublicKeyTemplate)
|
|
|
971d89 |
- free_attribute_array(new_pPublicKeyTemplate,
|
|
|
971d89 |
- new_ulPublicKeyAttributeCount);
|
|
|
971d89 |
if (new_publ_attrs)
|
|
|
971d89 |
free_attribute_array(new_publ_attrs, new_publ_attrs_len);
|
|
|
971d89 |
if (new_priv_attrs)
|
|
|
971d89 |
--
|
|
|
971d89 |
2.16.2.windows.1
|
|
|
971d89 |
|