[msan] Don't EXPECT_POISONED beyond the we_wordv

vitalybuka · vitalybuka · commit 4c699b1cd04d · 2021-08-24T17:19:10.000-07:00
Partially reverts commit 629411d. EXPECT_POISONED argument is outside of the allocation so we can't assume the state of shadow there.
diff --git a/compiler-rt/lib/msan/tests/msan_test.cpp b/compiler-rt/lib/msan/tests/msan_test.cpp
@@ -3758,7 +3758,6 @@ TEST(MemorySanitizer, wordexp) {
   ASSERT_STREQ("a", w.we_wordv[0]);
   ASSERT_STREQ("b", w.we_wordv[1]);
   ASSERT_STREQ("c", w.we_wordv[2]);
-  EXPECT_POISONED(w.we_wordv[3]);
 }
 
 TEST(MemorySanitizer, wordexp_initial_offset) {
@@ -3771,7 +3770,6 @@ TEST(MemorySanitizer, wordexp_initial_offset) {
   ASSERT_STREQ("a", w.we_wordv[1]);
   ASSERT_STREQ("b", w.we_wordv[2]);
   ASSERT_STREQ("c", w.we_wordv[3]);
-  EXPECT_POISONED(w.we_wordv[4]);
 }
 
 template<class T>

Original file line number	Diff line number	Diff line change
`@@ -3758,7 +3758,6 @@ TEST(MemorySanitizer, wordexp) {`
`3758`	`3758`	`ASSERT_STREQ("a", w.we_wordv[0]);`
`3759`	`3759`	`ASSERT_STREQ("b", w.we_wordv[1]);`
`3760`	`3760`	`ASSERT_STREQ("c", w.we_wordv[2]);`
`3761`		`- EXPECT_POISONED(w.we_wordv[3]);`
`3762`	`3761`	`}`
`3763`	`3762`
`3764`	`3763`	`TEST(MemorySanitizer, wordexp_initial_offset) {`
`@@ -3771,7 +3770,6 @@ TEST(MemorySanitizer, wordexp_initial_offset) {`
`3771`	`3770`	`ASSERT_STREQ("a", w.we_wordv[1]);`
`3772`	`3771`	`ASSERT_STREQ("b", w.we_wordv[2]);`
`3773`	`3772`	`ASSERT_STREQ("c", w.we_wordv[3]);`
`3774`		`- EXPECT_POISONED(w.we_wordv[4]);`
`3775`	`3773`	`}`
`3776`	`3774`
`3777`	`3775`	`template<class T>`