commit 3b1c072bfca1cd72b20a28e3f80868711deed650
author danh-arm <dan.handley@arm.com> Wed Apr 29 18:07:10 2020 +0100
committer GitHub <noreply@github.com> Wed Apr 29 18:07:10 2020 +0100
tree 0cd6c53e56b1c75c9988a87cf6e28e226ce3119d
parent 924a1f2f474a7e53344f245a6dee878674ec34bb
parent d3085ab2b856692960b0574013c28f36f8883bdf

Merge pull request #3217 from irwir/fix_ssl_tls.c

Avoid re-assigning zero to `ret` variable.

library/ssl_tls.c

diff --git a/library/ssl_tls.c b/library/ssl_tls.c
index dbc5a3e..ccfc4bd 100644
--- a/library/ssl_tls.c
+++ b/library/ssl_tls.c
@@ -2747,9 +2747,7 @@
     {
         ssl->session_negotiate->verify_result = MBEDTLS_X509_BADCERT_MISSING;
 
-        if( authmode == MBEDTLS_SSL_VERIFY_OPTIONAL )
-            ret = 0;
-        else
+        if( authmode != MBEDTLS_SSL_VERIFY_OPTIONAL )
             ret = MBEDTLS_ERR_SSL_NO_CLIENT_CERTIFICATE;
 
         goto exit;