Skip to content

Commit d0bf9c4

Browse files
committed
Revert "Disable exporting dilithium DER tests without WOLFSSL_ASN_TEMPLATE"
This reverts commit bbcdfe9.
1 parent 4064696 commit d0bf9c4

File tree

3 files changed

+4
-41
lines changed

3 files changed

+4
-41
lines changed

tests/api/test_mldsa.c

Lines changed: 1 addition & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -3004,13 +3004,8 @@ int test_wc_dilithium_der(void)
30043004

30053005
ExpectIntEQ(wc_Dilithium_PrivateKeyToDer(NULL, NULL,
30063006
0 ), WC_NO_ERR_TRACE(BAD_FUNC_ARG));
3007-
#ifndef WOLFSSL_ASN_TEMPLATE
3008-
ExpectIntEQ(wc_Dilithium_PrivateKeyToDer(key , NULL,
3009-
0 ), BAD_FUNC_ARG);
3010-
#else
30113007
ExpectIntGT(wc_Dilithium_PrivateKeyToDer(key , NULL,
30123008
0 ), 0);
3013-
#endif
30143009
ExpectIntEQ(wc_Dilithium_PrivateKeyToDer(NULL, der ,
30153010
0 ), WC_NO_ERR_TRACE(BAD_FUNC_ARG));
30163011
ExpectIntEQ(wc_Dilithium_PrivateKeyToDer(NULL, NULL,
@@ -3020,23 +3015,13 @@ int test_wc_dilithium_der(void)
30203015
ExpectIntEQ(wc_Dilithium_PrivateKeyToDer(key , der ,
30213016
0 ), WC_NO_ERR_TRACE(BUFFER_E));
30223017
/* Get length only. */
3023-
#ifndef WOLFSSL_ASN_TEMPLATE
3024-
ExpectIntEQ(wc_Dilithium_PrivateKeyToDer(key , NULL,
3025-
DILITHIUM_MAX_DER_SIZE), BAD_FUNC_ARG);
3026-
#else
30273018
ExpectIntEQ(wc_Dilithium_PrivateKeyToDer(key , NULL,
30283019
DILITHIUM_MAX_DER_SIZE), privDerLen);
3029-
#endif
30303020

30313021
ExpectIntEQ(wc_Dilithium_KeyToDer(NULL, NULL, 0 ),
30323022
WC_NO_ERR_TRACE(BAD_FUNC_ARG));
3033-
#ifndef WOLFSSL_ASN_TEMPLATE
3034-
ExpectIntEQ(wc_Dilithium_KeyToDer(key , NULL, 0 ),
3035-
BAD_FUNC_ARG);
3036-
#else
30373023
ExpectIntGT(wc_Dilithium_KeyToDer(key , NULL, 0 ),
30383024
0 );
3039-
#endif
30403025
ExpectIntEQ(wc_Dilithium_KeyToDer(NULL, der , 0 ),
30413026
WC_NO_ERR_TRACE(BAD_FUNC_ARG));
30423027
ExpectIntEQ(wc_Dilithium_KeyToDer(NULL, NULL, DILITHIUM_MAX_DER_SIZE),
@@ -3046,13 +3031,8 @@ int test_wc_dilithium_der(void)
30463031
ExpectIntEQ(wc_Dilithium_KeyToDer(key , der , 0 ),
30473032
WC_NO_ERR_TRACE(BUFFER_E));
30483033
/* Get length only. */
3049-
#ifndef WOLFSSL_ASN_TEMPLATE
3050-
ExpectIntEQ(wc_Dilithium_KeyToDer(key , NULL, DILITHIUM_MAX_DER_SIZE),
3051-
BAD_FUNC_ARG);
3052-
#else
30533034
ExpectIntEQ(wc_Dilithium_KeyToDer(key , NULL, DILITHIUM_MAX_DER_SIZE),
30543035
keyDerLen);
3055-
#endif
30563036

30573037
ExpectIntEQ(wc_Dilithium_PublicKeyDecode(NULL, NULL, NULL, 0 ),
30583038
WC_NO_ERR_TRACE(BAD_FUNC_ARG));
@@ -3101,34 +3081,22 @@ int test_wc_dilithium_der(void)
31013081
idx = 0;
31023082
ExpectIntEQ(wc_Dilithium_PublicKeyDecode(der, &idx, key, len), 0);
31033083

3104-
#ifndef WOLFSSL_ASN_TEMPLATE
3105-
ExpectIntEQ(len = wc_Dilithium_PrivateKeyToDer(key, der,
3106-
DILITHIUM_MAX_DER_SIZE), BAD_FUNC_ARG);
3107-
#else
31083084
ExpectIntEQ(len = wc_Dilithium_PrivateKeyToDer(key, der,
31093085
DILITHIUM_MAX_DER_SIZE), privDerLen);
31103086
idx = 0;
31113087
ExpectIntEQ(wc_Dilithium_PrivateKeyDecode(der, &idx, key, len), 0);
3112-
#endif
31133088

3114-
#ifndef WOLFSSL_ASN_TEMPLATE
3115-
ExpectIntEQ(len = wc_Dilithium_KeyToDer(key, der, DILITHIUM_MAX_DER_SIZE),
3116-
BAD_FUNC_ARG);
3117-
#else
31183089
ExpectIntEQ(len = wc_Dilithium_KeyToDer(key, der, DILITHIUM_MAX_DER_SIZE),
31193090
keyDerLen);
31203091
idx = 0;
31213092
ExpectIntEQ(wc_Dilithium_PrivateKeyDecode(der, &idx, key, len), 0);
3122-
#endif
31233093

31243094

31253095
wc_dilithium_free(key);
31263096
wc_FreeRng(&rng);
31273097

31283098
XFREE(der, NULL, DYNAMIC_TYPE_TMP_BUFFER);
31293099
XFREE(key, NULL, DYNAMIC_TYPE_TMP_BUFFER);
3130-
3131-
(void)keyDerLen;
31323100
#endif
31333101
return EXPECT_RESULT();
31343102
}
@@ -16910,7 +16878,7 @@ int test_mldsa_pkcs8_export_import_wolfSSL_form(void)
1691016878
(!defined(NO_WOLFSSL_CLIENT) || !defined(NO_WOLFSSL_SERVER)) && \
1691116879
!defined(WOLFSSL_DILITHIUM_NO_MAKE_KEY) && \
1691216880
!defined(WOLFSSL_DILITHIUM_NO_SIGN) && \
16913-
!defined(WOLFSSL_DILITHIUM_NO_ASN1) && defined(WOLFSSL_ASN_TEMPLATE)
16881+
!defined(WOLFSSL_DILITHIUM_NO_ASN1)
1691416882

1691516883
WOLFSSL_CTX* ctx = NULL;
1691616884
size_t i;

wolfcrypt/src/asn.c

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -37583,11 +37583,6 @@ int SetAsymKeyDer(const byte* privKey, word32 privKeyLen,
3758337583
}
3758437584

3758537585
#ifndef WOLFSSL_ASN_TEMPLATE
37586-
if (privKeyLen >= 128 || pubKeyLen >= 128) {
37587-
/* privKeyLen and pubKeyLen are assumed to be less than 128 */
37588-
return BAD_FUNC_ARG;
37589-
}
37590-
3759137586
/* calculate size */
3759237587
if (pubKey) {
3759337588
pubSz = 2 + pubKeyLen;

wolfcrypt/test/test.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46932,7 +46932,7 @@ static wc_test_ret_t test_dilithium_decode_level(const byte* rawKey,
4693246932
int isPublicOnlyKey)
4693346933
{
4693446934
int ret = 0;
46935-
#if !defined(WOLFSSL_DILITHIUM_NO_ASN1) && defined(WOLFSSL_ASN_TEMPLATE)
46935+
#ifndef WOLFSSL_DILITHIUM_NO_ASN1
4693646936
/* Size the buffer to accommodate the largest encoded key size */
4693746937
const word32 maxDerSz = DILITHIUM_MAX_PRV_KEY_DER_SIZE;
4693846938
word32 derSz;
@@ -46982,7 +46982,7 @@ static wc_test_ret_t test_dilithium_decode_level(const byte* rawKey,
4698246982
#endif
4698346983
}
4698446984

46985-
#if !defined(WOLFSSL_DILITHIUM_NO_ASN1) && defined(WOLFSSL_ASN_TEMPLATE)
46985+
#ifndef WOLFSSL_DILITHIUM_NO_ASN1
4698646986
/* Export raw key as DER */
4698746987
if (ret == 0) {
4698846988
#ifdef WOLFSSL_DILITHIUM_PUBLIC_KEY
@@ -47056,7 +47056,7 @@ static wc_test_ret_t test_dilithium_decode_level(const byte* rawKey,
4705647056
ret = WC_TEST_RET_ENC_NC;
4705747057
}
4705847058
#endif /* !WOLFSSL_DILITHIUM_FIPS204_DRAFT */
47059-
#endif /* !WOLFSSL_DILITHIUM_NO_ASN1 && WOLFSSL_ASN_TEMPLATE */
47059+
#endif /* WOLFSSL_DILITHIUM_NO_ASN1 */
4706047060

4706147061
/* Cleanup */
4706247062
wc_dilithium_free(key);

0 commit comments

Comments
 (0)