this loop isn't necessary. It's done by xlat_copy.

diff --git a/src/lib/unlang/xlat_tokenize.c b/src/lib/unlang/xlat_tokenize.c
index 9f69a5195b758e5afe13b4c60bdab8721f420a59..a39c7f2e19b95bf77eff9ea4253184730416b948 100644 (file)
--- a/src/lib/unlang/xlat_tokenize.c
+++ b/src/lib/unlang/xlat_tokenize.c
@@ -126,13 +126,7 @@ xlat_exp_t *xlat_exp_func_alloc(TALLOC_CTX *ctx, xlat_t *func, xlat_exp_head_t c
                return NULL;
        }
        node->flags = func->flags;
-
-       /*
-        *      A pure function can have impure arguments, e.g. hash(sql query).
-        */
-       xlat_exp_foreach(args, arg) {
-               xlat_flags_merge(&node->flags, &arg->flags);
-       }
+       xlat_flags_merge(&node->flags, &args->flags);
 
        return node;
 }