mirror of
https://github.com/Mbed-TLS/mbedtls.git
synced 2026-03-20 19:21:09 +01:00
Switch to a default value of -1u
Since we explicitly document the value 0xFFFFFFFF or -1u as representing 'result not available', we can use it as a sensible default value without creating an API change. Use this value instead of introducing a new verification result value. Signed-off-by: David Horstmann <david.horstmann@arm.com>
This commit is contained in:
@@ -108,7 +108,6 @@
|
||||
#define MBEDTLS_X509_BADCRL_BAD_MD 0x020000 /**< The CRL is signed with an unacceptable hash. */
|
||||
#define MBEDTLS_X509_BADCRL_BAD_PK 0x040000 /**< The CRL is signed with an unacceptable PK alg (eg RSA vs ECDSA). */
|
||||
#define MBEDTLS_X509_BADCRL_BAD_KEY 0x080000 /**< The CRL is signed with an unacceptable key (eg bad curve, RSA too short). */
|
||||
#define MBEDTLS_X509_VERIFY_NOT_STARTED 0x100000 /**< No verification has yet been performed (used as a safe initial value). */
|
||||
|
||||
/** \} name X509 Verify codes */
|
||||
/** \} addtogroup x509_module */
|
||||
|
||||
@@ -209,10 +209,7 @@ mbedtls_x509_crt_profile;
|
||||
"The CRL is signed with an unacceptable PK alg (eg RSA vs ECDSA).") \
|
||||
X509_CRT_ERROR_INFO(MBEDTLS_X509_BADCRL_BAD_KEY, \
|
||||
"MBEDTLS_X509_BADCRL_BAD_KEY", \
|
||||
"The CRL is signed with an unacceptable key (eg bad curve, RSA too short).") \
|
||||
X509_CRT_ERROR_INFO(MBEDTLS_X509_VERIFY_NOT_STARTED, \
|
||||
"MBEDTLS_X509_VERIFY_NOT_STARTED", \
|
||||
"No verification has yet been performed.")
|
||||
"The CRL is signed with an unacceptable key (eg bad curve, RSA too short).")
|
||||
|
||||
/**
|
||||
* Container for writing a certificate (CRT)
|
||||
|
||||
@@ -1048,8 +1048,8 @@ void mbedtls_ssl_transform_init(mbedtls_ssl_transform *transform)
|
||||
void mbedtls_ssl_session_init(mbedtls_ssl_session *session)
|
||||
{
|
||||
memset(session, 0, sizeof(mbedtls_ssl_session));
|
||||
/* Set verify_result to indicate failure by default. */
|
||||
session->verify_result = MBEDTLS_X509_VERIFY_NOT_STARTED;
|
||||
/* Set verify_result to -1u to indicate 'result not available'. */
|
||||
session->verify_result = 0xFFFFFFFF;
|
||||
}
|
||||
|
||||
MBEDTLS_CHECK_RETURN_CRITICAL
|
||||
|
||||
@@ -6024,7 +6024,7 @@ void verify_result_without_handshake(void)
|
||||
|
||||
uint32_t verify_result = mbedtls_ssl_get_verify_result(&ssl);
|
||||
|
||||
TEST_EQUAL(verify_result, MBEDTLS_X509_VERIFY_NOT_STARTED);
|
||||
TEST_EQUAL(verify_result, 0xFFFFFFFF);
|
||||
|
||||
exit:
|
||||
mbedtls_ssl_config_free(&conf);
|
||||
|
||||
Reference in New Issue
Block a user