diff options
author | Android Build Coastguard Worker <android-build-coastguard-worker@google.com> | 2022-03-23 10:05:22 +0000 |
---|---|---|
committer | Android Build Coastguard Worker <android-build-coastguard-worker@google.com> | 2022-03-23 10:05:22 +0000 |
commit | a9438b7e146dc478609fd9d6e8f7d6dcb453316b (patch) | |
tree | adbbfac0d7c5c098c1eca004477f93adefc158a9 /tests/malloc_test.cpp | |
parent | 23ac20ab05ecba77fcabc7e34cfdb1b3c757417e (diff) | |
parent | f3d2af29a7965555d717798faee043c536b86b2f (diff) |
Snap for 8343869 from f3d2af29a7965555d717798faee043c536b86b2f to s-keystone-qcom-release
Change-Id: I6a04b5d19eefd1210fe07399797fceb909010cb6
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); + } + } +} |