commit d3085ab2b856692960b0574013c28f36f8883bdf
author irwir <irwir@users.noreply.github.com> Tue Apr 21 22:26:05 2020 +0300
committer irwir <irwir@users.noreply.github.com> Tue Apr 21 22:26:59 2020 +0300
tree 9efb4d0e5c545c0879c220f1dce5176c10be27ea
parent 5cac33722878e51b764d571e40a627264f922b43

Avoid re-assigning zero to `ret` variable.
Resolve #3182.

Signed-off-by: irwir <irwir@users.noreply.github.com>

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;