manually free the pair list, as done in the other test functions.

otherwise with the new tlist code, the pairs are freed via
the talloc_free_children() callback, and the PAIR_VERIFY() check
fails, because the pair is still in a list.

diff --git a/src/bin/unit_test_attribute.c b/src/bin/unit_test_attribute.c
index 125bac9d8cf2675060a57695b5dd8196c26ea00a..fb9686b18622df565fbd4902b301cd6b617f577f 100644 (file)
--- a/src/bin/unit_test_attribute.c
+++ b/src/bin/unit_test_attribute.c
@@ -2343,6 +2343,8 @@ static size_t command_pair(command_result_t *result, command_file_ctx_t *cc,
                }
        }
 
+       PAIR_LIST_VERIFY(&head);
+
        p = data;
        end = data + COMMAND_OUTPUT_MAX;
        for (vp = fr_pair_list_head(&head);
@@ -2363,6 +2365,8 @@ static size_t command_pair(command_result_t *result, command_file_ctx_t *cc,
        if (p > data) p -= 2;
        *p = 0;
 
+       fr_pair_list_free(&head);
+
        RETURN_OK(p - data);
 }