commit aa544796aee46919530f53474b60c8be3c9108c0
author Peter Collingbourne <pcc@google.com> Thu May 13 13:53:37 2021 -0700
committer Peter Collingbourne <pcc@google.com> Thu May 13 14:08:41 2021 -0700
tree 62c776fc1f071e5a25e0c3bdefab20fcbe0d527e
parent afe3212a194f45565aff963858c8d412f71b50d7

Run MTE tests on zero-sized allocations.

Bug: 187879470
Change-Id: I9547f3032af9d1a921f8597a53389d25af33b369

debuggerd/debuggerd_test.cpp

diff --git a/debuggerd/debuggerd_test.cpp b/debuggerd/debuggerd_test.cpp
index 144faee..013c572 100644
--- a/debuggerd/debuggerd_test.cpp
+++ b/debuggerd/debuggerd_test.cpp
@@ -473,7 +473,7 @@
 
 struct SizeParamCrasherTest : CrasherTest, testing::WithParamInterface<size_t> {};
 
-INSTANTIATE_TEST_SUITE_P(Sizes, SizeParamCrasherTest, testing::Values(16, 131072));
+INSTANTIATE_TEST_SUITE_P(Sizes, SizeParamCrasherTest, testing::Values(0, 16, 131072));
 
 TEST_P(SizeParamCrasherTest, mte_uaf) {
 #if defined(__aarch64__)
@@ -481,6 +481,11 @@
     GTEST_SKIP() << "Requires MTE";
   }
 
+  // Any UAF on a zero-sized allocation will be out-of-bounds so it won't be reported.
+  if (GetParam() == 0) {
+    return;
+  }
+
   int intercept_result;
   unique_fd output_fd;
   StartProcess([&]() {