diff mbox series

[Concept,23/33] test: dm: Dispose tree in livetree_ensure()

Message ID	20260416023021.626949-24-sjg@u-boot.org
State	New
Headers	From: Simon Glass <sjg@u-boot.org> To: U-Boot Concept <concept@u-boot.org> Date: Wed, 15 Apr 2026 20:29:48 -0600 Message-ID: <20260416023021.626949-24-sjg@u-boot.org> In-Reply-To: <20260416023021.626949-1-sjg@u-boot.org> References: <20260416023021.626949-1-sjg@u-boot.org> MIME-Version: 1.0 Message-ID-Hash: A2HVJE6Q6QIZTARC7T6VIK5AG2LUX2US CC: Simon Glass <sjg@chromium.org> Precedence: list Subject: [Concept] [PATCH 23/33] test: dm: Dispose tree in livetree_ensure() Archived-At: <https://lists.u-boot.org/archives/list/concept@u-boot.org/message/A2HVJE6Q6QIZTARC7T6VIK5AG2LUX2US/> Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit
Series	Fix memory leaks and test pollution in sandbox tests \| [Concept,00/33] Fix memory leaks and test pollution in sandbox tests [Concept,01/33] malloc: Double the allocation registry size [Concept,02/33] malloc: Expose registry-overflow state to tests [Concept,03/33] malloc: Report mcheck-registry entry count [Concept,04/33] malloc: Expose the largest-contiguous free region [Concept,05/33] malloc: Find aligned allocations using the registry [Concept,06/33] virtio: Free mmio base when removing sandbox virtio emul [Concept,07/33] virtio: Delete vqs when removing a device [Concept,08/33] virtio: emul_blk: Free disk data on remove [Concept,09/33] usb: Drain finished scan threads after run_threads [Concept,10/33] blk: Mark partition name as alloced [Concept,11/33] test: common: Target largest free chunk in heap-pressure tests [Concept,12/33] test: Save and restore global DM state around UTF_DM tests [Concept,13/33] test: video: Restore auto-sync at end of video_sync_damage [Concept,14/33] test: video: Reselect default font at start of video_silence [Concept,15/33] test: usb: Reset usb_started at start of usb_stop test [Concept,16/33] boot: Close the filesystem after reading a bootflow [Concept,17/33] boot: Free the logo buffer when freeing a bootflow [Concept,18/33] bootctl: Destroy the UI expo when the device is removed [Concept,19/33] bootctl: simple_state: Free items alist on remove [Concept,20/33] boot: Replace the current cedit expo on reload [Concept,21/33] boot: cedit: Leave manual-sync off with exit without save [Concept,22/33] upl: Dispose of live trees after use [Concept,23/33] test: dm: Dispose tree in livetree_ensure() [Concept,24/33] test: dm: Dispose tree in oftree_new [Concept,25/33] test: dm: Free trees in ofnode_too_many() [Concept,26/33] test: dm: Release resources in oftree_to_fdt(() test [Concept,27/33] test: upl: Dispose oftrees in upl_test_base() [Concept,28/33] test: lib: Dispose tree in json_to_fdt_luks2() [Concept,29/33] test: boot: Clean up DM state after bootflow_cmdline() [Concept,30/33] test: boot: Clean up DM state after bootstd_images() [Concept,31/33] test: boot: Clean up DM state after bootstd_adhoc() [Concept,32/33] test: setexpr: Unset test env var in str_long() [Concept,33/33] test: Clear bootstd cur_bootflow after each test

Commit Message

Simon Glass April 16, 2026, 2:29 a.m. UTC

  From: Simon Glass <sjg@chromium.org>

This dm_test_oftree_new() test unflattens a secondary device tree using
oftree_from_fdt() but never frees it, thus leaking about 6K of memory
per run.

Dispose of the tree before returning.

Signed-off-by: Simon Glass <sjg@chromium.org>
---

 test/dm/ofnode.c | 2 ++
 1 file changed, 2 insertions(+)

diff mbox series

Patch

diff --git a/test/dm/ofnode.c b/test/dm/ofnode.c
index 0f60c2a6281..516e24ceb28 100644
--- a/test/dm/ofnode.c
+++ b/test/dm/ofnode.c
@@ -1492,6 +1492,8 @@  static int dm_test_livetree_ensure(struct unit_test_state *uts)
 	ut_asserteq_str("sandbox-other2",
 			ofnode_read_string(node, "compatible"));
 
+	oftree_dispose(tree);
+
 	return 0;
 }
 DM_TEST(dm_test_livetree_ensure, UTF_SCAN_FDT);