diff mbox series

[Concept,10/33] blk: Mark partition name as alloced

Message ID	20260416023021.626949-11-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:35 -0600 Message-ID: <20260416023021.626949-11-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: PYGFFCFPBIDFV6TC346GDLOPLBU5NIUX CC: Simon Glass <sjg@chromium.org> Precedence: list Subject: [Concept] [PATCH 10/33] blk: Mark partition name as alloced Archived-At: <https://lists.u-boot.org/archives/list/concept@u-boot.org/message/PYGFFCFPBIDFV6TC346GDLOPLBU5NIUX/> 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>

part_create_block_devices() allocates a name for the block-partition
driver, but device_bind_driver() does not set DM_FLAG_NAME_ALLOCED, so
the name is not released when the partition device is unbound. Every
scanned partition thus leaks its name.

Call device_set_name_alloced() so the core frees it on unbind.

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

 drivers/block/blk-uclass.c | 1 +
 1 file changed, 1 insertion(+)

diff mbox series

Patch

diff --git a/drivers/block/blk-uclass.c b/drivers/block/blk-uclass.c
index 9c9169d6ac8..611859435f2 100644
--- a/drivers/block/blk-uclass.c
+++ b/drivers/block/blk-uclass.c
@@ -808,6 +808,7 @@  static int part_create_block_devices(struct udevice *blk_dev)
 					 strdup(devname), &dev);
 		if (ret)
 			return ret;
+		device_set_name_alloced(dev);
 
 		part_data = dev_get_uclass_plat(dev);
 		part_data->partnum = part;