diff mbox series

[Concept,25/35] sandbox: Enable mcheck heap protection

Message ID	20251210000737.180797-26-sjg@u-boot.org
State	New
Headers	From: Simon Glass <sjg@u-boot.org> To: U-Boot Concept <concept@u-boot.org> Date: Tue, 9 Dec 2025 17:07:16 -0700 Message-ID: <20251210000737.180797-26-sjg@u-boot.org> In-Reply-To: <20251210000737.180797-1-sjg@u-boot.org> References: <20251210000737.180797-1-sjg@u-boot.org> MIME-Version: 1.0 Message-ID-Hash: Q3OCO3ERXCHMW5YNOBGC5L243Y7K2JDW CC: Heinrich Schuchardt <xypron.glpk@gmx.de>, Simon Glass <simon.glass@canonical.com>, Claude <noreply@anthropic.com> Precedence: list Subject: [Concept] [PATCH 25/35] sandbox: Enable mcheck heap protection Archived-At: <https://lists.u-boot.org/archives/list/concept@u-boot.org/message/Q3OCO3ERXCHMW5YNOBGC5L243Y7K2JDW/> Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit
Series	malloc: Add heap debugging commands and mcheck caller tracking \| [Concept,00/35] malloc: Add heap debugging commands and mcheck caller tracking [Concept,01/35] watchdog: Unregister cyclic on device removal [Concept,03/35] test: dm: Fix memory leaks in test drivers [Concept,04/35] test: dm: Fix delta usage in dm_test_devres_free() [Concept,05/35] mmc: sandbox: Fix memory leak on probe failure [Concept,06/35] video: truetype: Use pre-allocated buffer for glyph rendering [Concept,07/35] lib: Add format_size() to format sizes as strings [Concept,08/35] test: Add ut_asserteq_regex() for regex pattern matching [Concept,09/35] test: Show the required size when console-record overflows [Concept,10/35] malloc: Use mcheck.h header instead of local declaration [Concept,11/35] malloc: Make mcheck respect REALLOC_ZERO_BYTES_FREES [Concept,12/35] backtrace: Use a static buffer in backtrace_ctx for symbols [Concept,13/35] backtrace: Add backtrace_str() for condensed backtrace output [Concept,14/35] malloc: Fix unused internal_memalign warning with mcheck [Concept,15/35] malloc: Add malloc_get_info() to retrieve memory statistics [Concept,16/35] malloc: Add a Kconfig option for debugging malloc() [Concept,17/35] malloc: Enable stats if UNIT_TEST or MALLOC_DEBUG [Concept,18/35] malloc: Add 'malloc' command with 'info' subcommand [Concept,19/35] malloc: Add call counters for malloc, free, realloc [Concept,20/35] malloc: Add a Kconfig option for heap protection [Concept,21/35] malloc: Move mcheck block after includes [Concept,22/35] malloc: Fix internal calls and memalign for mcheck [Concept,23/35] malloc: Update the valloc functions to use mcheck wrappers [Concept,24/35] malloc: Increase the mcheck registry size [Concept,25/35] sandbox: Enable mcheck heap protection [Concept,26/35] malloc: Remove warning messages during relocation [Concept,27/35] malloc: Support storing caller information [Concept,28/35] malloc: Add a caller-info parameter to dlmalloc_impl() [Concept,29/35] malloc: Add malloc dump command to walk the heap [Concept,30/35] malloc: Fix malloc_dump to find mcheck headers in memalign chunks [Concept,31/35] malloc: Record caller backtrace for each allocation [Concept,32/35] malloc: Skip backtrace when stack is corrupted [Concept,33/35] malloc: Show caller info for freed chunks in malloc_dump [Concept,34/35] malloc: Print mcheck registry-overflow message only once [Concept,35/35] doc: malloc: Document debugging features

Commit Message

Simon Glass Dec. 10, 2025, 12:07 a.m. UTC

  From: Simon Glass <simon.glass@canonical.com>

Enable CONFIG_MCHECK_HEAP_PROTECTION for the sandbox board to detect
heap corruption during development and testing.

Increase the heap size from 96 MiB to 128 MiB to account for the
additional overhead of mcheck metadata (~48 bytes per allocation).

Increase the console-out buffer to 128K to allow space for the new
malloc dump, when checked in tests.

Co-developed-by: Claude <noreply@anthropic.com>
Signed-off-by: Simon Glass <simon.glass@canonical.com>
---

 configs/sandbox_defconfig | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff mbox series

Patch

diff --git a/configs/sandbox_defconfig b/configs/sandbox_defconfig
index 44185ccb7eb..b1b2e725b4c 100644
--- a/configs/sandbox_defconfig
+++ b/configs/sandbox_defconfig
@@ -1,5 +1,5 @@ 
 CONFIG_TEXT_BASE=0
-CONFIG_SYS_MALLOC_LEN=0x6000000
+CONFIG_SYS_MALLOC_LEN=0x8000000
 CONFIG_BLOBLIST_SIZE_RELOC=0x20000
 CONFIG_NR_DRAM_BANKS=1
 CONFIG_ENV_SIZE=0x2000
@@ -12,6 +12,7 @@  CONFIG_DEBUG_UART=y
 CONFIG_SYS_MEMTEST_START=0x00100000
 CONFIG_SYS_MEMTEST_END=0x00101000
 CONFIG_ULIB=y
+CONFIG_MCHECK_HEAP_PROTECTION=y
 CONFIG_EXAMPLES=y
 CONFIG_EFI_SECURE_BOOT=y
 CONFIG_EFI_RT_VOLATILE_STORE=y
@@ -49,7 +50,7 @@  CONFIG_IMAGE_PRE_LOAD=y
 CONFIG_IMAGE_PRE_LOAD_SIG=y
 CONFIG_CEDIT=y
 CONFIG_CONSOLE_RECORD=y
-CONFIG_CONSOLE_RECORD_OUT_SIZE=0x6000
+CONFIG_CONSOLE_RECORD_OUT_SIZE=0x20000
 CONFIG_PRE_CONSOLE_BUFFER=y
 CONFIG_LOG=y
 CONFIG_LOG_MAX_LEVEL=9