[coreboot-gerrit] New patch to review for coreboot: 0c071cb libpayload: Set heap's header size to 64-bit

Patrick Georgi (patrick@georgi-clan.de) gerrit at coreboot.org
Fri Aug 30 15:42:37 CEST 2013


Patrick Georgi (patrick at georgi-clan.de) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3889

-gerrit

commit 0c071cb7d0b10d4510f9ec3200e1e4e9d9bdc376
Author: Nico Huber <nico.huber at secunet.com>
Date:   Fri Jun 14 15:34:59 2013 +0200

    libpayload: Set heap's header size to 64-bit
    
    Change-Id: Ic82d6477c53da62a1325400f2e596d7d557d5d1e
    Signed-off-by: Patrick Georgi <patrick.georgi at secunet.com>
---
 payloads/libpayload/libc/malloc.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/payloads/libpayload/libc/malloc.c b/payloads/libpayload/libc/malloc.c
index c03cc73..851ef57 100644
--- a/payloads/libpayload/libc/malloc.c
+++ b/payloads/libpayload/libc/malloc.c
@@ -37,9 +37,6 @@
  * We're also susceptible to the usual buffer overrun poisoning, though the
  * risk is within acceptable ranges for this implementation (don't overrun
  * your buffers, kids!).
- *
- * The header format (hdrtype_t) supports heaps of up to 32MiB (given that int
- * is 32 bits long).
  */
 
 #define IN_MALLOC_C
@@ -50,7 +47,7 @@ extern char _heap, _eheap;	/* Defined in the ldscript. */
 static void *hstart = (void *)&_heap;
 static void *hend = (void *)&_eheap;
 
-typedef unsigned int hdrtype_t;
+typedef u64 hdrtype_t;
 #define HDRSIZE (sizeof(hdrtype_t))
 
 #define SIZE_BITS ((HDRSIZE << 3) - 7)



More information about the coreboot-gerrit mailing list