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 ea4ca780311ff57ff94e54bb639a73db4bb2b0b5..082dc9d991a681655d4cf9ba37f27be1213a80bc 100644 (file)
--- 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;
 }