From 5b5420573cd6c78e1a87eaa4946e13b9150f9076 Mon Sep 17 00:00:00 2001 From: Adriaan de Jong Date: Sat, 4 Feb 2012 14:16:13 +0100 Subject: [PATCH] Minor code cleanup: cleaned up error handling in verify_cert. Removed done label and cleaned up return values. Signed-off-by: Adriaan de Jong Acked-by: David Sommerseth Signed-off-by: David Sommerseth --- ssl_verify.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/ssl_verify.c b/ssl_verify.c index 0b2a1fb94..e45f149c3 100644 --- a/ssl_verify.c +++ b/ssl_verify.c @@ -669,14 +669,14 @@ verify_cert(struct tls_session *session, x509_cert_t *cert, int cert_depth) msg (D_HANDSHAKE, "VERIFY OK: depth=%d, %s", cert_depth, subject); session->verified = true; - done: x509_free_subject (subject); - return (session->verified == true) ? SUCCESS : FAILURE; + return SUCCESS; err: tls_clear_error(); session->verified = false; - goto done; + x509_free_subject (subject); + return FAILURE; } /* *************************************************************************** -- 2.47.2