diff options
author | Android Build Coastguard Worker <android-build-coastguard-worker@google.com> | 2021-11-19 00:01:00 +0000 |
---|---|---|
committer | Android Build Coastguard Worker <android-build-coastguard-worker@google.com> | 2021-11-19 00:01:00 +0000 |
commit | 11fa611f2c52e17e23f679869724278e2becc862 (patch) | |
tree | 9d0fa3e8914b318787dc8e2f87a09fe56c2489f0 /tests/malloc_test.cpp | |
parent | ae52060801a5c4a8bc508a4c5b603a8d29681fc3 (diff) | |
parent | 1ec0946834b56d44fe69256ad6bbbe72f551b991 (diff) |
Snap for 7929330 from 1ec0946834b56d44fe69256ad6bbbe72f551b991 to sc-v2-release
Change-Id: Iea6c8d6a7d6a8a0a15ca5748b82989ad50034dd6
Diffstat (limited to 'tests/malloc_test.cpp')
-rw-r--r-- | tests/malloc_test.cpp | 22 |
1 files changed, 22 insertions, 0 deletions
diff --git a/tests/malloc_test.cpp b/tests/malloc_test.cpp index d73f2436d..f4a1c0d30 100644 --- a/tests/malloc_test.cpp +++ b/tests/malloc_test.cpp @@ -1371,3 +1371,25 @@ TEST(malloc, allocation_slack) { GTEST_SKIP() << "bionic extension"; #endif } + +// Regression test for b/206701345 -- scudo bug, MTE only. +// Fix: https://reviews.llvm.org/D105261 +// Fix: https://android-review.googlesource.com/c/platform/external/scudo/+/1763655 +TEST(malloc, realloc_mte_crash_b206701345) { + // We want to hit in-place realloc at the very end of an mmap-ed region. Not + // all size classes allow such placement - mmap size has to be divisible by + // the block size. At the time of writing this could only be reproduced with + // 64 byte size class (i.e. 48 byte allocations), but that may change in the + // future. Try several different classes at the lower end. + std::vector<void*> ptrs(10000); + for (int i = 1; i < 32; ++i) { + size_t sz = 16 * i - 1; + for (void*& p : ptrs) { + p = realloc(malloc(sz), sz + 1); + } + + for (void* p : ptrs) { + free(p); + } + } +} |