Update.
authorUlrich Drepper <drepper@redhat.com>
Sat, 20 Nov 2004 04:45:06 +0000 (04:45 +0000)
committerUlrich Drepper <drepper@redhat.com>
Sat, 20 Nov 2004 04:45:06 +0000 (04:45 +0000)
* malloc/malloc.c (_int_malloc): Check for corruption of chunk
which is about to be returned.

ChangeLog
malloc/malloc.c

index ae8cc2e..a5cd019 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,8 @@
 2004-11-19  Ulrich Drepper  <drepper@redhat.com>
 
+       * malloc/malloc.c (_int_malloc): Check for corruption of chunk
+       which is about to be returned.
+
        * malloc/malloc.c (_int_free): Add a few more cheap tests for
        corruption.
 
index d6810be..b62ffb5 100644 (file)
@@ -3840,8 +3840,12 @@ _int_malloc(mstate av, size_t bytes)
   */
 
   if ((unsigned long)(nb) <= (unsigned long)(av->max_fast)) {
-    fb = &(av->fastbins[(fastbin_index(nb))]);
+    long int idx = fastbin_index(nb);
+    fb = &(av->fastbins[idx]);
     if ( (victim = *fb) != 0) {
+      if (__builtin_expect (fastbin_index (chunksize (victim)) != idx, 0))
+       malloc_printerr (check_action, "malloc(): memory corruption (fast)",
+                        chunk2mem (victim));
       *fb = victim->fd;
       check_remalloced_chunk(av, victim, nb);
       return chunk2mem(victim);
@@ -3911,6 +3915,10 @@ _int_malloc(mstate av, size_t bytes)
 
     while ( (victim = unsorted_chunks(av)->bk) != unsorted_chunks(av)) {
       bck = victim->bk;
+      if (__builtin_expect (victim->size <= 2 * SIZE_SZ, 0)
+         || __builtin_expect (victim->size > av->system_mem, 0))
+       malloc_printerr (check_action, "malloc(): memory corruption",
+                        chunk2mem (victim));
       size = chunksize(victim);
 
       /*