Skip to content

Commit dbe724f

Browse files
committed
Minor test fixes and version improvements
1 parent f37b8db commit dbe724f

File tree

4 files changed

+25
-10
lines changed

4 files changed

+25
-10
lines changed

src/wolfpkcs11.c

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -26,9 +26,10 @@
2626
#include <wolfpkcs11/pkcs11.h>
2727
#include <wolfpkcs11/internal.h>
2828

29-
/* Function list table. */
29+
/* Function list table for PKCS#11 v2.40 */
3030
static CK_FUNCTION_LIST wolfpkcs11FunctionList = {
31-
{ 2, 40 },
31+
/* Version: Major, Minor */
32+
{ 2, 40 },
3233

3334
C_Initialize,
3435
C_Finalize,
@@ -104,8 +105,10 @@ static CK_FUNCTION_LIST wolfpkcs11FunctionList = {
104105

105106
CK_RV C_GetInfoV3_0(CK_INFO_PTR pInfo);
106107

108+
/* Function list table for PKCS#11 v3.0 */
107109
static CK_FUNCTION_LIST_3_0 wolfpkcs11FunctionList_3_0 = {
108-
{ 3, 0 },
110+
/* Version: Major, Minor */
111+
{ 3, 0 },
109112

110113
C_Initialize,
111114
C_Finalize,
@@ -207,8 +210,10 @@ static CK_FUNCTION_LIST_3_0 wolfpkcs11FunctionList_3_0 = {
207210

208211
CK_RV C_GetInfoV3_2(CK_INFO_PTR pInfo);
209212

213+
/* Function list table for PKCS#11 v3.2 */
210214
static CK_FUNCTION_LIST_3_2 wolfpkcs11FunctionList_3_2 = {
211-
{ CRYPTOKI_VERSION_MAJOR, CRYPTOKI_VERSION_MINOR },
215+
/* Version: Major, Minor */
216+
{ 3, 2 },
212217

213218
C_Initialize,
214219
C_Finalize,

tests/pkcs11mtt.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1523,7 +1523,7 @@ static CK_RV test_recover(void* args)
15231523
}
15241524
if (ret == CKR_OK) {
15251525
ret = funcList->C_VerifyRecover(session, sig, sigSz, data, &dataSz);
1526-
#ifndef NO_RSA
1526+
#if !defined(NO_RSA) && defined(WC_RSA_DIRECT)
15271527
CHECK_CKR_FAIL(ret, CKR_OPERATION_NOT_INITIALIZED,
15281528
"Verify Recover not initialized");
15291529
#else

tests/pkcs11test.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5353,7 +5353,7 @@ static CK_RV test_recover(void* args)
53535353
}
53545354
if (ret == CKR_OK) {
53555355
ret = funcList->C_VerifyRecover(session, sig, sigSz, data, &dataSz);
5356-
#ifndef NO_RSA
5356+
#if !defined(NO_RSA) && defined(WC_RSA_DIRECT)
53575357
CHECK_CKR_FAIL(ret, CKR_OPERATION_NOT_INITIALIZED,
53585358
"Verify Recover not initialized");
53595359
#else
@@ -5365,7 +5365,7 @@ static CK_RV test_recover(void* args)
53655365
return ret;
53665366
}
53675367

5368-
#ifndef NO_RSA
5368+
#if !defined(NO_RSA) && defined(WC_RSA_DIRECT)
53695369
static CK_RV rsa_verify_recover(CK_SESSION_HANDLE session,
53705370
CK_MECHANISM_TYPE mech_type)
53715371
{
@@ -15793,7 +15793,7 @@ static TEST_FUNC testFunc[] = {
1579315793
PKCS11TEST_FUNC_SESS_DECL(test_digest_fail),
1579415794
PKCS11TEST_FUNC_SESS_DECL(test_sign_verify),
1579515795
PKCS11TEST_FUNC_SESS_DECL(test_recover),
15796-
#ifndef NO_RSA
15796+
#if !defined(NO_RSA) && defined(WC_RSA_DIRECT)
1579715797
PKCS11TEST_FUNC_SESS_DECL(test_verify_recover_pkcs),
1579815798
PKCS11TEST_FUNC_SESS_DECL(test_verify_recover_x509),
1579915799
#endif

wolfpkcs11/pkcs11.h

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,8 +73,18 @@ extern "C" {
7373

7474
#define CK_NULL_PTR NULL_PTR
7575

76-
#define CRYPTOKI_VERSION_MAJOR 3
77-
#define CRYPTOKI_VERSION_MINOR 2
76+
#ifndef CRYPTOKI_VERSION_MAJOR
77+
#if defined(WOLFPKCS11_PKCS11_V3_2)
78+
#define CRYPTOKI_VERSION_MAJOR 3
79+
#define CRYPTOKI_VERSION_MINOR 2
80+
#elif defined(WOLFPKCS11_PKCS11_V3_0)
81+
#define CRYPTOKI_VERSION_MAJOR 3
82+
#define CRYPTOKI_VERSION_MINOR 0
83+
#else
84+
#define CRYPTOKI_VERSION_MAJOR 2
85+
#define CRYPTOKI_VERSION_MINOR 40
86+
#endif
87+
#endif
7888

7989

8090
#define CK_INVALID_HANDLE 0UL

0 commit comments

Comments
 (0)