a7a908e
Only in b/malloc: arena.c.orig
a7a908e
Only in b/malloc: hooks.c.orig
a7a908e
diff -rup a/malloc/malloc.c b/malloc/malloc.c
a7a908e
--- a/malloc/malloc.c	2012-02-14 10:08:22.062534892 -0700
a7a908e
+++ b/malloc/malloc.c	2012-02-14 10:19:43.088724473 -0700
a7a908e
@@ -2936,8 +2936,9 @@ public_mALLOc(size_t bytes)
a7a908e
       (void)mutex_unlock(&ar_ptr->mutex);
a7a908e
     } else {
a7a908e
       /* ... or sbrk() has failed and there is still a chance to mmap() */
a7a908e
-      ar_ptr = arena_get2(ar_ptr->next ? ar_ptr : 0, bytes);
a7a908e
-      (void)mutex_unlock(&main_arena.mutex);
a7a908e
+      mstate prev = ar_ptr->next ? ar_ptr : 0;
a7a908e
+      (void)mutex_unlock(&ar_ptr->mutex);
a7a908e
+      ar_ptr = arena_get2(prev, bytes);
a7a908e
       if(ar_ptr) {
a7a908e
 	victim = _int_malloc(ar_ptr, bytes);
a7a908e
 	(void)mutex_unlock(&ar_ptr->mutex);
a7a908e
@@ -3151,23 +3152,26 @@ public_vALLOc(size_t bytes)
a7a908e
   if(!ar_ptr)
a7a908e
     return 0;
a7a908e
   p = _int_valloc(ar_ptr, bytes);
a7a908e
-  (void)mutex_unlock(&ar_ptr->mutex);
a7a908e
   if(!p) {
a7a908e
     /* Maybe the failure is due to running out of mmapped areas. */
a7a908e
     if(ar_ptr != &main_arena) {
a7a908e
+      (void)mutex_unlock(&ar_ptr->mutex);
a7a908e
       ar_ptr = &main_arena;
a7a908e
       (void)mutex_lock(&ar_ptr->mutex);
a7a908e
       p = _int_memalign(ar_ptr, pagesz, bytes);
a7a908e
       (void)mutex_unlock(&ar_ptr->mutex);
a7a908e
     } else {
a7a908e
       /* ... or sbrk() has failed and there is still a chance to mmap() */
a7a908e
-      ar_ptr = arena_get2(ar_ptr->next ? ar_ptr : 0, bytes);
a7a908e
+      mstate prev = ar_ptr->next ? ar_ptr : 0;
a7a908e
+      (void)mutex_unlock(&ar_ptr->mutex);
a7a908e
+      ar_ptr = arena_get2(prev, bytes);
a7a908e
       if(ar_ptr) {
a7a908e
 	p = _int_memalign(ar_ptr, pagesz, bytes);
a7a908e
 	(void)mutex_unlock(&ar_ptr->mutex);
a7a908e
       }
a7a908e
     }
a7a908e
-  }
a7a908e
+  } else
a7a908e
+    (void)mutex_unlock(&ar_ptr->mutex);
a7a908e
   assert(!p || chunk_is_mmapped(mem2chunk(p)) ||
a7a908e
 	 ar_ptr == arena_for_chunk(mem2chunk(p)));
a7a908e
 
a7a908e
@@ -3195,24 +3199,26 @@ public_pVALLOc(size_t bytes)
a7a908e
 
a7a908e
   arena_get(ar_ptr, bytes + 2*pagesz + MINSIZE);
a7a908e
   p = _int_pvalloc(ar_ptr, bytes);
a7a908e
-  (void)mutex_unlock(&ar_ptr->mutex);
a7a908e
   if(!p) {
a7a908e
     /* Maybe the failure is due to running out of mmapped areas. */
a7a908e
     if(ar_ptr != &main_arena) {
a7a908e
+      (void)mutex_unlock(&ar_ptr->mutex);
a7a908e
       ar_ptr = &main_arena;
a7a908e
       (void)mutex_lock(&ar_ptr->mutex);
a7a908e
       p = _int_memalign(ar_ptr, pagesz, rounded_bytes);
a7a908e
       (void)mutex_unlock(&ar_ptr->mutex);
a7a908e
     } else {
a7a908e
       /* ... or sbrk() has failed and there is still a chance to mmap() */
a7a908e
-      ar_ptr = arena_get2(ar_ptr->next ? ar_ptr : 0,
a7a908e
-			  bytes + 2*pagesz + MINSIZE);
a7a908e
+      mstate prev = ar_ptr->next ? ar_ptr : 0;
a7a908e
+      (void)mutex_unlock(&ar_ptr->mutex);
a7a908e
+      ar_ptr = arena_get2(prev, bytes + 2*pagesz + MINSIZE);
a7a908e
       if(ar_ptr) {
a7a908e
 	p = _int_memalign(ar_ptr, pagesz, rounded_bytes);
a7a908e
 	(void)mutex_unlock(&ar_ptr->mutex);
a7a908e
       }
a7a908e
     }
a7a908e
-  }
a7a908e
+  } else
a7a908e
+    (void)mutex_unlock(&ar_ptr->mutex);
a7a908e
   assert(!p || chunk_is_mmapped(mem2chunk(p)) ||
a7a908e
 	 ar_ptr == arena_for_chunk(mem2chunk(p)));
a7a908e
 
a7a908e
@@ -3277,8 +3283,6 @@ public_cALLOc(size_t n, size_t elem_size
a7a908e
 #endif
a7a908e
   mem = _int_malloc(av, sz);
a7a908e
 
a7a908e
-  /* Only clearing follows, so we can unlock early. */
a7a908e
-  (void)mutex_unlock(&av->mutex);
a7a908e
 
a7a908e
   assert(!mem || chunk_is_mmapped(mem2chunk(mem)) ||
a7a908e
 	 av == arena_for_chunk(mem2chunk(mem)));
a7a908e
@@ -3286,21 +3290,23 @@ public_cALLOc(size_t n, size_t elem_size
a7a908e
   if (mem == 0) {
a7a908e
     /* Maybe the failure is due to running out of mmapped areas. */
a7a908e
     if(av != &main_arena) {
a7a908e
+      (void)mutex_unlock(&av->mutex);
a7a908e
       (void)mutex_lock(&main_arena.mutex);
a7a908e
       mem = _int_malloc(&main_arena, sz);
a7a908e
       (void)mutex_unlock(&main_arena.mutex);
a7a908e
     } else {
a7a908e
       /* ... or sbrk() has failed and there is still a chance to mmap() */
a7a908e
-      (void)mutex_lock(&main_arena.mutex);
a7a908e
-      av = arena_get2(av->next ? av : 0, sz);
a7a908e
-      (void)mutex_unlock(&main_arena.mutex);
a7a908e
+      mstate prev = av->next ? av : 0;
a7a908e
+      (void)mutex_unlock(&av->mutex);
a7a908e
+      av = arena_get2(prev, sz);
a7a908e
       if(av) {
a7a908e
 	mem = _int_malloc(av, sz);
a7a908e
 	(void)mutex_unlock(&av->mutex);
a7a908e
       }
a7a908e
     }
a7a908e
     if (mem == 0) return 0;
a7a908e
-  }
a7a908e
+  } else
a7a908e
+    (void)mutex_unlock(&av->mutex);
a7a908e
   p = mem2chunk(mem);
a7a908e
 
a7a908e
   /* Two optional cases in which clearing not necessary */