e1b411d
diff -Nrup a/malloc/arena.c b/malloc/arena.c
e1b411d
--- a/malloc/arena.c	2012-05-29 16:45:53.000000000 -0600
e1b411d
+++ b/malloc/arena.c	2012-05-30 00:13:40.683514016 -0600
e1b411d
@@ -673,7 +673,7 @@ heap_trim(heap_info *heap, size_t pad)
2716093
     heap = prev_heap;
2716093
     if(!prev_inuse(p)) { /* consolidate backward */
2716093
       p = prev_chunk(p);
2716093
-      unlink(p, bck, fwd);
2716093
+      unlink(ar_ptr, p, bck, fwd);
2716093
     }
2716093
     assert(((unsigned long)((char*)p + new_size) & (pagesz-1)) == 0);
2716093
     assert( ((char*)p + new_size) == ((char*)heap + heap->size) );
e1b411d
diff -Nrup a/malloc/hooks.c b/malloc/hooks.c
e1b411d
--- a/malloc/hooks.c	2012-05-29 16:45:53.000000000 -0600
e1b411d
+++ b/malloc/hooks.c	2012-05-30 00:13:40.684514011 -0600
e1b411d
@@ -191,7 +191,9 @@ top_check(void)
2716093
 	(char*)t + chunksize(t) == mp_.sbrk_base + main_arena.system_mem)))
2716093
     return 0;
2716093
 
2716093
+  mutex_unlock(&main_arena);
2716093
   malloc_printerr (check_action, "malloc: top chunk is corrupt", t);
2716093
+  mutex_lock(&main_arena);
2716093
 
2716093
   /* Try to set up a new top chunk. */
2716093
   brk = MORECORE(0);
e1b411d
diff -Nrup a/malloc/malloc.c b/malloc/malloc.c
e1b411d
--- a/malloc/malloc.c	2012-05-29 16:45:53.000000000 -0600
e1b411d
+++ b/malloc/malloc.c	2012-05-30 00:13:40.686514001 -0600
e1b411d
@@ -1424,12 +1424,14 @@ typedef struct malloc_chunk* mbinptr;
2716093
 #define last(b)      ((b)->bk)
2716093
 
2716093
 /* Take a chunk off a bin list */
2716093
-#define unlink(P, BK, FD) {                                            \
2716093
+#define unlink(AV, P, BK, FD) {					       \
2716093
   FD = P->fd;                                                          \
2716093
   BK = P->bk;                                                          \
2716093
-  if (__builtin_expect (FD->bk != P || BK->fd != P, 0))                \
2716093
+  if (__builtin_expect (FD->bk != P || BK->fd != P, 0)) {	       \
2716093
+    mutex_unlock(&(AV)->mutex);					       \
2716093
     malloc_printerr (check_action, "corrupted double-linked list", P); \
2716093
-  else {                                                               \
2716093
+    mutex_lock(&(AV)->mutex);					       \
2716093
+  } else {							       \
2716093
     FD->bk = BK;                                                       \
2716093
     BK->fd = FD;                                                       \
2716093
     if (!in_smallbin_range (P->size)				       \
e1b411d
@@ -2511,7 +2513,9 @@ static void* sysmalloc(INTERNAL_SIZE_T n
2716093
 
2716093
     else if (contiguous(av) && old_size && brk < old_end) {
2716093
       /* Oops!  Someone else killed our space..  Can't touch anything.  */
2716093
+      mutex_unlock(&av->mutex);
2716093
       malloc_printerr (3, "break adjusted to free malloc space", brk);
2716093
+      mutex_lock(&av->mutex);
2716093
     }
2716093
 
2716093
     /*
e1b411d
@@ -3345,7 +3349,9 @@ _int_malloc(mstate av, size_t bytes)
2716093
 	{
2716093
 	  errstr = "malloc(): memory corruption (fast)";
2716093
 	errout:
2716093
+	  mutex_unlock(&av->mutex);
2716093
 	  malloc_printerr (check_action, errstr, chunk2mem (victim));
2716093
+	  mutex_lock(&av->mutex);
2716093
 	  return NULL;
2716093
 	}
2716093
       check_remalloced_chunk(av, victim, nb);
e1b411d
@@ -3430,8 +3436,12 @@ _int_malloc(mstate av, size_t bytes)
2716093
       bck = victim->bk;
2716093
       if (__builtin_expect (victim->size <= 2 * SIZE_SZ, 0)
2716093
 	  || __builtin_expect (victim->size > av->system_mem, 0))
2716093
-	malloc_printerr (check_action, "malloc(): memory corruption",
2716093
-			 chunk2mem (victim));
2716093
+	{
2716093
+	  void *p = chunk2mem(victim);
2716093
+	  mutex_unlock(&av->mutex);
2716093
+	  malloc_printerr (check_action, "malloc(): memory corruption", p);
2716093
+	  mutex_lock(&av->mutex);
2716093
+	}
2716093
       size = chunksize(victim);
2716093
 
2716093
       /*
e1b411d
@@ -3572,7 +3582,7 @@ _int_malloc(mstate av, size_t bytes)
2716093
 	  victim = victim->fd;
2716093
 
2716093
 	remainder_size = size - nb;
2716093
-	unlink(victim, bck, fwd);
2716093
+	unlink(av, victim, bck, fwd);
2716093
 
2716093
 	/* Exhaust */
2716093
 	if (remainder_size < MINSIZE)  {
e1b411d
@@ -3670,7 +3680,7 @@ _int_malloc(mstate av, size_t bytes)
2716093
 	remainder_size = size - nb;
2716093
 
2716093
 	/* unlink */
2716093
-	unlink(victim, bck, fwd);
2716093
+	unlink(av, victim, bck, fwd);
2716093
 
2716093
 	/* Exhaust */
2716093
 	if (remainder_size < MINSIZE) {
e1b411d
@@ -3805,9 +3815,11 @@ _int_free(mstate av, mchunkptr p, int ha
2716093
     {
2716093
       errstr = "free(): invalid pointer";
2716093
     errout:
2716093
-      if (! have_lock && locked)
2716093
+      if (have_lock || locked)
2716093
 	(void)mutex_unlock(&av->mutex);
2716093
       malloc_printerr (check_action, errstr, chunk2mem(p));
2716093
+      if (have_lock)
2716093
+	mutex_lock(&av->mutex);
2716093
       return;
2716093
     }
e1b411d
   /* We know that each chunk is at least MINSIZE bytes in size or a
e1b411d
@@ -3952,7 +3964,7 @@ _int_free(mstate av, mchunkptr p, int ha
2716093
       prevsize = p->prev_size;
2716093
       size += prevsize;
2716093
       p = chunk_at_offset(p, -((long) prevsize));
2716093
-      unlink(p, bck, fwd);
2716093
+      unlink(av, p, bck, fwd);
2716093
     }
2716093
 
2716093
     if (nextchunk != av->top) {
e1b411d
@@ -3961,7 +3973,7 @@ _int_free(mstate av, mchunkptr p, int ha
2716093
 
2716093
       /* consolidate forward */
2716093
       if (!nextinuse) {
2716093
-	unlink(nextchunk, bck, fwd);
2716093
+	unlink(av, nextchunk, bck, fwd);
2716093
 	size += nextsize;
2716093
       } else
2716093
 	clear_inuse_bit_at_offset(nextchunk, 0);
e1b411d
@@ -4122,7 +4134,7 @@ static void malloc_consolidate(mstate av
2716093
 	    prevsize = p->prev_size;
2716093
 	    size += prevsize;
2716093
 	    p = chunk_at_offset(p, -((long) prevsize));
2716093
-	    unlink(p, bck, fwd);
2716093
+	    unlink(av, p, bck, fwd);
2716093
 	  }
2716093
 
2716093
 	  if (nextchunk != av->top) {
e1b411d
@@ -4130,7 +4142,7 @@ static void malloc_consolidate(mstate av
2716093
 
2716093
 	    if (!nextinuse) {
2716093
 	      size += nextsize;
2716093
-	      unlink(nextchunk, bck, fwd);
2716093
+	      unlink(av, nextchunk, bck, fwd);
2716093
 	    } else
2716093
 	      clear_inuse_bit_at_offset(nextchunk, 0);
2716093
 
e1b411d
@@ -4199,7 +4211,9 @@ _int_realloc(mstate av, mchunkptr oldp,
2716093
     {
2716093
       errstr = "realloc(): invalid old size";
2716093
     errout:
2716093
+      mutex_unlock(&av->mutex);
2716093
       malloc_printerr (check_action, errstr, chunk2mem(oldp));
2716093
+      mutex_lock(&av->mutex);
2716093
       return NULL;
2716093
     }
2716093
 
e1b411d
@@ -4241,7 +4255,7 @@ _int_realloc(mstate av, mchunkptr oldp,
2716093
 	     (unsigned long)(newsize = oldsize + nextsize) >=
2716093
 	     (unsigned long)(nb)) {
2716093
       newp = oldp;
2716093
-      unlink(next, bck, fwd);
2716093
+      unlink(av, next, bck, fwd);
2716093
     }
2716093
 
2716093
     /* allocate, copy, free */