Skip to content

Commit

Permalink
Fail with NOT_COMPILED_IN if someone tries to use ConfirmSignature wi…
Browse files Browse the repository at this point in the history
…th NO_ASN_CRYPT. Also default to signature failed.
  • Loading branch information
dgarske committed Aug 2, 2024
1 parent a18d016 commit 25d14f1
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions wolfcrypt/src/asn.c
Original file line number Diff line number Diff line change
Expand Up @@ -16612,7 +16612,7 @@ static int ConfirmSignature(SignatureCtx* sigCtx,
const byte* sigParams, word32 sigParamsSz,
byte* rsaKeyIdx)
{
int ret = 0;
int ret = ASN_SIG_CONFIRM_E; /* default to failure */
#if defined(WOLFSSL_RENESAS_TSIP_TLS) || defined(WOLFSSL_RENESAS_FSPSM_TLS)
CertAttribute* certatt = NULL;
#endif
Expand Down Expand Up @@ -17749,8 +17749,8 @@ static int ConfirmSignature(SignatureCtx* sigCtx,
exit_cs:

#else
/* Warning: The NO_ASN_CRYPT option skips signature checking! */
ret = 0; /* allow unchecked signature */
/* For NO_ASN_CRYPT return "not compiled in" */
ret = NOT_COMPILED_IN;
#endif /* !NO_ASN_CRYPT */

(void)keyOID;
Expand Down

0 comments on commit 25d14f1

Please sign in to comment.