label should always be defined

diff --git a/src/main/client.c b/src/main/client.c
index e4691ed2bdf726ec47657c72572be72df435a509..408ea74be19c1b84d8f6eb7d4de6c13806a58364 100644 (file)
--- 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