]> git.dujemihanovic.xyz Git - u-boot.git/commit
dlmalloc: calloc: fix zeroing early allocations
authorSimon Goldschmidt <simon.k.r.goldschmidt@gmail.com>
Fri, 25 Oct 2019 19:23:35 +0000 (21:23 +0200)
committerTom Rini <trini@konsulko.com>
Thu, 7 Nov 2019 23:01:13 +0000 (18:01 -0500)
commitbb71a2d9dcd9c53aa4d4b8e4d26c24d9b59b74c3
tree7f289d63b28dca3ffa38463c850f4b00738ada29
parentd024e992b3f88745ca80ce81055974c9bf2134d4
dlmalloc: calloc: fix zeroing early allocations

When full malloc is enabled and SYS_MALLOC_F is also enabled, the simple
pre-reloc heap is used before relocation. In this case, calloc() uses
the MALLOC_ZERO macro to zero out the allocated memory. However, since
this macro is specially crafted for the dlmalloc implementation, it
does not always work for simple malloc.

For example, when allocating 16 bytes via simple malloc, only the first
12 bytes get zeroed out. The last 4 bytes will remain untouched.

This is a problem for DM drivers that are allocated before relocation:
memory allocated via 'platdata_auto_alloc_size' might not be set to
zero, resulting in bogus behaviour.

To fix this, use 'memset' instead of 'MALLOC_ZERO' to zero out memory
that compes from simple malloc.

Signed-off-by: Simon Goldschmidt <simon.k.r.goldschmidt@gmail.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
common/dlmalloc.c