From: Arran Cudbard-Bell <a.cudbardb@freeradius.org>
Date: Thu, 14 Dec 2023 04:00:57 +0000 (-0600)
Subject: Stop processing isn't really correct here, and leads to spurious errors
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=5fb82e6cd7041e60b02c549174567d6be309382d;p=thirdparty%2Ffreeradius-server.git

Stop processing isn't really correct here, and leads to spurious errors
---

diff --git a/src/lib/unlang/module.c b/src/lib/unlang/module.c
index ea4ca780311..082dc9d991a 100644
--- a/src/lib/unlang/module.c
+++ b/src/lib/unlang/module.c
@@ -508,9 +508,7 @@ unlang_action_t unlang_module_yield_to_tmpl(TALLOC_CTX *ctx, fr_value_box_list_t
 	/*
 	 *	Push the xlat function
 	 */
-	if (unlang_tmpl_push(ctx, out, request, vpt, args) < 0) {
-		return UNLANG_ACTION_STOP_PROCESSING;
-	}
+	if (unlang_tmpl_push(ctx, out, request, vpt, args) < 0) return UNLANG_ACTION_FAIL;
 
 	return UNLANG_ACTION_PUSHED_CHILD;
 }