we shouldn't be copying structural groups.

We may change this later, but for now let's disallow it until all
of the tests pass.

diff --git a/src/lib/unlang/xlat_eval.c b/src/lib/unlang/xlat_eval.c
index 66d1ea56e2a180957cd2eeee9d76b6dbd9bb99aa..95dfe091f2bb296fa53232588673c62b52c2af53 100644 (file)
--- a/src/lib/unlang/xlat_eval.c
+++ b/src/lib/unlang/xlat_eval.c
@@ -950,6 +950,7 @@ xlat_eval_pair_real(TALLOC_CTX *ctx, fr_value_box_list_t *out, request_t *reques
                value = fr_value_box_alloc(ctx, vp->data.type, vp->da, vp->data.tainted);
                if (!value) goto oom;
 
+               fr_assert(fr_type_is_leaf(vp->da->type));
                fr_value_box_copy(value, value, &vp->data);     /* Also dups taint */
                fr_dlist_insert_tail(out, value);
                break;