The default case statement meant we always returned... but I guess the compiler isn...

diff --git a/src/modules/rlm_crl/rlm_crl.c b/src/modules/rlm_crl/rlm_crl.c
index e0df79771dee5a2c412e0208b3bec66c5e0bb943..bdc732468a38b486faf75704ffdc9446cf1abbef 100644 (file)
--- 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)