about summary refs log tree commit diff
diff options
context:
space:
mode:
authorUlrich Drepper <drepper@redhat.com>2004-09-09 17:15:57 +0000
committerUlrich Drepper <drepper@redhat.com>2004-09-09 17:15:57 +0000
commit14abadc8bf54ed8cf3585aa2f0aa869df4f83983 (patch)
tree6b1910ba64ac38cd9eb52b9056155271930d8331
parent6bf4302e764e422656f4f7230619fe9ab481ae04 (diff)
downloadglibc-14abadc8bf54ed8cf3585aa2f0aa869df4f83983.tar.gz
glibc-14abadc8bf54ed8cf3585aa2f0aa869df4f83983.tar.xz
glibc-14abadc8bf54ed8cf3585aa2f0aa869df4f83983.zip
Update.
	* malloc/malloc.c (_int_free): Add inexpensive double free and
	memory corruption tests.
	(malloc_printf_nc): Renamed to malloc_printerr.  Second parameter
	is no format string anymore.  Don't use stdio.  Adjust all callers.
	* malloc/hooks.c: Adjust malloc_printf_nc callers.
-rw-r--r--ChangeLog6
-rw-r--r--malloc/hooks.c8
2 files changed, 9 insertions, 5 deletions
diff --git a/ChangeLog b/ChangeLog
index f0a74a85df..c0868ed174 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -6,7 +6,11 @@
 
 	* sysdeps/unix/sysv/linux/bits/socket.h (__SCM_CONNECT): Removed.
 
-	* malloc/malloc.c (_int_free): Add inexpensive double free test.
+	* malloc/malloc.c (_int_free): Add inexpensive double free and
+	memory corruption tests.
+	(malloc_printf_nc): Renamed to malloc_printerr.  Second parameter
+	is no format string anymore.  Don't use stdio.  Adjust all callers.
+	* malloc/hooks.c: Adjust malloc_printf_nc callers.
 
 2004-09-08  Roland McGrath  <roland@redhat.com>
 
diff --git a/malloc/hooks.c b/malloc/hooks.c
index ebb37de969..4de558270f 100644
--- a/malloc/hooks.c
+++ b/malloc/hooks.c
@@ -99,7 +99,7 @@ __malloc_check_init()
   __realloc_hook = realloc_check;
   __memalign_hook = memalign_check;
   if(check_action & 1)
-    malloc_printf_nc (1, "malloc: using debugging hooks\n");
+    malloc_printerr (5, "malloc: using debugging hooks", NULL);
 }
 
 /* A simple, standard set of debugging hooks.  Overhead is `only' one
@@ -216,7 +216,7 @@ top_check()
   if((char*)t + chunksize(t) == mp_.sbrk_base + main_arena.system_mem ||
      t == initial_top(&main_arena)) return 0;
 
-  malloc_printf_nc (check_action, "malloc: top chunk is corrupt\n");
+  malloc_printerr (check_action | 4, "malloc: top chunk is corrupt", NULL);
 
   /* Try to set up a new top chunk. */
   brk = MORECORE(0);
@@ -268,7 +268,7 @@ free_check(mem, caller) Void_t* mem; const Void_t *caller;
   if(!p) {
     (void)mutex_unlock(&main_arena.mutex);
 
-    malloc_printf_nc(check_action, "free(): invalid pointer %p!\n", mem);
+    malloc_printerr(check_action, "free(): invalid pointer", mem);
     return;
   }
 #if HAVE_MMAP
@@ -302,7 +302,7 @@ realloc_check(oldmem, bytes, caller)
   oldp = mem2chunk_check(oldmem);
   (void)mutex_unlock(&main_arena.mutex);
   if(!oldp) {
-    malloc_printf_nc(check_action, "realloc(): invalid pointer %p!\n", oldmem);
+    malloc_printerr(check_action, "realloc(): invalid pointer", oldmem);
     return malloc_check(bytes, NULL);
   }
   oldsize = chunksize(oldp);