diff options
author | Christopher Ferris <cferris@google.com> | 2019-03-22 18:23:31 -0700 |
---|---|---|
committer | android-build-merger <android-build-merger@google.com> | 2019-03-22 18:23:31 -0700 |
commit | 08d6c0f8458c9380d9c404d55fea17176d588afa (patch) | |
tree | 24de748bcd50dbc19bab882b77d0c5a636ae3513 | |
parent | b4e426ffebd74493d341945490d68f8bbc38e7c5 (diff) | |
parent | 46971b81ee021c2ca19da8d65bdca8a61669ce41 (diff) | |
download | platform_external_jemalloc_new-08d6c0f8458c9380d9c404d55fea17176d588afa.tar.gz platform_external_jemalloc_new-08d6c0f8458c9380d9c404d55fea17176d588afa.tar.bz2 platform_external_jemalloc_new-08d6c0f8458c9380d9c404d55fea17176d588afa.zip |
Fix bug trying to examine unused extents.
am: 46971b81ee
Change-Id: I2c52b0d542aa1c273857486d312675f63625a6dc
-rw-r--r-- | src/android_je_iterate.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/src/android_je_iterate.c b/src/android_je_iterate.c index f0a4cc3e..ca776e00 100644 --- a/src/android_je_iterate.c +++ b/src/android_je_iterate.c @@ -34,6 +34,12 @@ int je_iterate(uintptr_t base, size_t size, continue; } + if (extent_szind_get_maybe_invalid(extent) >= NSIZES) { + // Ignore this unused extent. + ptr = (uintptr_t)extent_past_get(extent); + continue; + } + szind_t szind; bool slab; rtree_szind_slab_read(tsd_tsdn(tsd), &extents_rtree, rtree_ctx, ptr, true, &szind, &slab); @@ -56,8 +62,7 @@ int je_iterate(uintptr_t base, size_t size, callback(allocated_ptr, bin_size, arg); } } - } else if (extent_state_get(extent) == extent_state_active && - extent_szind_get_maybe_invalid(extent) < NSIZES) { + } else if (extent_state_get(extent) == extent_state_active) { // Large allocation. uintptr_t base_ptr = (uintptr_t)extent_addr_get(extent); if (ptr <= base_ptr) { |