From: Simon Glass <sjg@chromium.org> The bootflow contains fields which are connected to the struct. It should be freed when it is finished with. It isn't exactly clear why this fixes the problem, which shows up as: => ut -f bootstd vbe_test_abrec_oem_norun Test: bootstd_setup_for_tests: bootstd_common.c common/dlmalloc.c:793: do_check_free_chunk: Assertion `next->prev_size == sz' failed. Fix it by adding a bootflow_free() at the end, which is good practice in any case. Signed-off-by: Simon Glass <sjg@chromium.org> Fixes: 07f5b96985d ("boot: test: Add a test for the VBE OS flow") --- test/boot/vbe_abrec_os.c | 1 + 1 file changed, 1 insertion(+) diff --git a/test/boot/vbe_abrec_os.c b/test/boot/vbe_abrec_os.c index 3ac95d7b4c3..34e96378574 100644 --- a/test/boot/vbe_abrec_os.c +++ b/test/boot/vbe_abrec_os.c @@ -115,6 +115,7 @@ static int check_abrec_norun(struct unit_test_state *uts, bool use_oem, root = oftree_root(tree); ut_asserteq_str("snow", ofnode_read_string(root, "compatible")); + bootflow_free(&bflow); return 0; } -- 2.43.0