From: Arran Cudbard-Bell <a.cudbardb@freeradius.org>
Date: Wed, 28 May 2025 14:31:18 +0000 (-0600)
Subject: The default case statement meant we always returned... but I guess the compiler isn... 
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=effd8ea3082c22b0ee70f954c888ec5e40758e7e;p=thirdparty%2Ffreeradius-server.git

The default case statement meant we always returned... but I guess the compiler isn't smart enough to realise that.
---

diff --git a/src/modules/rlm_crl/rlm_crl.c b/src/modules/rlm_crl/rlm_crl.c
index e0df79771de..bdc732468a3 100644
--- a/src/modules/rlm_crl/rlm_crl.c
+++ b/src/modules/rlm_crl/rlm_crl.c
@@ -293,8 +293,10 @@ static unlang_action_t crl_process_cdp_data(rlm_rcode_t *p_result, module_ctx_t
 
 	default:
 		REDEBUG("Too many CRL values returned, failing");
-		goto fail;
+		break;
 	}
+
+	goto fail;
 }
 
 static unlang_action_t crl_by_url(rlm_rcode_t *p_result, module_ctx_t const *mctx, request_t *request)