From: Alan T. DeKok <aland@freeradius.org>
Date: Thu, 9 Jul 2015 17:24:22 +0000 (-0400)
Subject: label should always be defined
X-Git-Tag: release_3_0_10~367
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=cf5eec91fdc0d383f90e398555967f71232cf6ba;p=thirdparty%2Ffreeradius-server.git

label should always be defined
---

diff --git a/src/main/client.c b/src/main/client.c
index e4691ed2bdf..408ea74be19 100644
--- a/src/main/client.c
+++ b/src/main/client.c
@@ -560,6 +560,8 @@ RADCLIENT_LIST *client_list_parse_section(CONF_SECTION *section, UNUSED bool tls
 	     cs = cf_subsection_find_next(section, cs, "client")) {
 		c = client_afrom_cs(cs, cs, in_server, false);
 		if (!c) {
+		error:
+			client_free(c);
 			client_list_free(clients);
 			return NULL;
 		}
@@ -571,10 +573,7 @@ RADCLIENT_LIST *client_list_parse_section(CONF_SECTION *section, UNUSED bool tls
 		 */
 		if (tls_required != c->tls_required) {
 			cf_log_err_cs(cs, "Client does not have the same TLS configuration as the listener");
-		error:
-			client_free(c);
-			client_list_free(clients);
-			return NULL;
+			goto error;
 		}
 #endif