From a5fb313cb7b7e692fd4684916aaa98e03ec7e8b6 Mon Sep 17 00:00:00 2001 From: Andreas Schwab Date: Mon, 14 Nov 2011 11:41:52 +0100 Subject: [PATCH] Don't call reused_arena when _int_new_arena failed --- ChangeLog | 5 +++++ malloc/arena.c | 10 +++++----- 2 files changed, 10 insertions(+), 5 deletions(-) diff --git a/ChangeLog b/ChangeLog index 8612676..cae28ff 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +2011-11-14 Andreas Schwab + + * malloc/arena.c (arena_get2): Don't call reused_arena when + _int_new_arena failed. + 2011-11-12 Ulrich Drepper * sysdeps/unix/clock_gettime.c (clock_gettime): No need to assign diff --git a/malloc/arena.c b/malloc/arena.c index 042cac8..cb8548b 100644 --- a/malloc/arena.c +++ b/malloc/arena.c @@ -844,14 +844,14 @@ arena_get2(mstate a_tsd, size_t size) size_t n = narenas; if (__builtin_expect (n <= mp_.arena_test || n < narenas_limit, 0)) { - if (catomic_compare_and_exchange_bool_acq(&narenas, n + 1, n)) + if (catomic_compare_and_exchange_bool_acq (&narenas, n + 1, n)) goto repeat; a = _int_new_arena (size); - if (__builtin_expect (a != NULL, 1)) - return a; - catomic_decrement(&narenas); + if (__builtin_expect (a == NULL, 0)) + catomic_decrement (&narenas); } - a = reused_arena (); + else + a = reused_arena (); } #else if(!a_tsd) -- 2.7.4