Commit Diff

Commit:
f31c3a3aa283bd6a34386e3c2fcdcad9ab36e5c4
From:
Florian Westphal <fw@strlen.de>
Date:
Message:
SSL/TLS: fix error path in gnutls ssl ctx allocation
Actions:
Patch | Tree
commit - b0931f322b527f24215419e9c73eda7d7f22de6a
commit + f31c3a3aa283bd6a34386e3c2fcdcad9ab36e5c4
blob - fbf796c79ab3f41da09269abb1d615f06810eb64
blob + ae1646c5571fb13da78362837330d48b245cdae5
--- src/ngircd/conn-ssl.c
+++ 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