From: Florian Westphal <fw@strlen.de>
Date: Sun, 17 May 2009 19:18:04 +0000 (+0200)
Subject: SSL/TLS: fix error path in gnutls ssl ctx allocation
X-Git-Tag: rel-15-rc1~50
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=f31c3a3aa283bd6a34386e3c2fcdcad9ab36e5c4;p=ngircd

SSL/TLS: fix error path in gnutls ssl ctx allocation
---

diff --git a/src/ngircd/conn-ssl.c b/src/ngircd/conn-ssl.c
index fbf796c7..ae1646c5 100644
--- a/src/ngircd/conn-ssl.c
+++ b/src/ngircd/conn-ssl.c
@@ -406,6 +406,7 @@ ConnSSL_Init_SSL(CONNECTION *c)
 	if (ret < 0) {
 		Log(LOG_ERR, "gnutls_set_default_priority: %s", gnutls_strerror(ret));
 		ConnSSL_Free(c);
+		return false;
 	}
 	/*
 	 * The intermediate (long) cast is here to avoid a warning like:
@@ -418,6 +419,7 @@ ConnSSL_Init_SSL(CONNECTION *c)
 	if (ret < 0) {
 		Log(LOG_ERR, "gnutls_credentials_set: %s", gnutls_strerror(ret));
 		ConnSSL_Free(c);
+		return false;
 	}
 	gnutls_dh_set_prime_bits(c->ssl_state.gnutls_session, DH_BITS);
 #endif