Merge pull request #187 from Yubico/pvs_remove_warnings

Remove some warnings
This commit is contained in:
Alessio Di Mauro
2019-03-06 15:27:40 +01:00
committed by GitHub
+1 -9
View File
@@ -1695,7 +1695,7 @@ CK_DEFINE_FUNCTION(CK_RV, C_SignInit)(
CK_KEY_TYPE type = 0;
CK_ULONG key_len = 0;
CK_BYTE exp[3];
CK_BYTE buf[1024];
CK_BYTE buf[1024] = {0};
CK_ATTRIBUTE template[] = {
{CKA_KEY_TYPE, &type, sizeof(type)},
{CKA_MODULUS_BITS, &key_len, sizeof(key_len)},
@@ -1863,12 +1863,6 @@ CK_DEFINE_FUNCTION(CK_RV, C_Sign)(
goto sign_out;
}
if (op_info.type != YKCS11_SIGN) {
DBG("Operation not initialized");
rv = CKR_OPERATION_NOT_INITIALIZED;
goto sign_out;
}
if (session.info.state == CKS_RO_PUBLIC_SESSION ||
session.info.state == CKS_RW_PUBLIC_SESSION) {
DBG("User is not logged in");
@@ -1974,8 +1968,6 @@ CK_DEFINE_FUNCTION(CK_RV, C_Sign)(
memcpy(pSignature, op_info.buf, *pulSignatureLen);
op_info.type = YKCS11_NOOP;
rv = CKR_OK;
sign_out: