Only return VERIFY_FAILED from a single point
Everything else is a fatal error. Also improve documentation about that for
the vrfy callback.
diff --git a/library/x509_crt.c b/library/x509_crt.c
index 70ad356..fb09253 100644
--- a/library/x509_crt.c
+++ b/library/x509_crt.c
@@ -1957,8 +1957,8 @@
/* path_cnt is 0 for the first intermediate CA */
if( 1 + path_cnt > POLARSSL_X509_MAX_INTERMEDIATE_CA )
{
- *flags |= BADCERT_NOT_TRUSTED;
- return( POLARSSL_ERR_X509_CERT_VERIFY_FAILED );
+ /* return immediately as the goal is to avoid unbounded recursion */
+ return( POLARSSL_ERR_X509_FATAL_ERROR );
}
if( x509_time_expired( &child->valid_to ) )
@@ -2174,6 +2174,10 @@
}
exit:
+ /* prevent misuse of the vrfy callback */
+ if( ret == POLARSSL_ERR_X509_CERT_VERIFY_FAILED )
+ ret = POLARSSL_ERR_X509_FATAL_ERROR;
+
if( ret != 0 )
{
*flags = (uint32_t) -1;