]> granicus.if.org Git - libvpx/commitdiff
test/vp[89]_boolcoder_test: quiet msan warnings
authorJames Zern <jzern@google.com>
Wed, 6 Nov 2019 07:02:44 +0000 (23:02 -0800)
committerJames Zern <jzern@google.com>
Wed, 6 Nov 2019 07:04:06 +0000 (23:04 -0800)
the bitreaders may fill beyond what was written to the buffer as an
optimization. the data isn't used meaningfully, but it may trigger a
msan warning.

BUG=b/140939146

Change-Id: Id03cd203b8ee7ecaf6fdfe3f3c9f2ccfec527129

test/vp8_boolcoder_test.cc
test/vp9_boolcoder_test.cc

index 9d81f9382ad22fd088835284c33617f90c80809e..c78b0b3b6cc25f6bfe93659f6dfd7929c9d5e7b6 100644 (file)
@@ -93,6 +93,9 @@ TEST(VP8, TestBitIO) {
         }
 
         vp8_stop_encode(&bw);
+        // vp8dx_bool_decoder_fill() may read into uninitialized data that
+        // isn't used meaningfully, but may trigger an MSan warning.
+        memset(bw_buffer + bw.pos, 0, sizeof(VP8_BD_VALUE) - 1);
 
         BOOL_DECODER br;
         encrypt_buffer(bw_buffer, kBufferSize);
index 5dbfd5ca59123c2c25762af106fd69e5b2c75271..0cafa6730e251a07ad682aa93d7b144c49bc05c3 100644 (file)
@@ -66,6 +66,9 @@ TEST(VP9, TestBitIO) {
         }
 
         vpx_stop_encode(&bw);
+        // vpx_reader_fill() may read into uninitialized data that
+        // isn't used meaningfully, but may trigger an MSan warning.
+        memset(bw_buffer + bw.pos, 0, sizeof(BD_VALUE) - 1);
 
         // First bit should be zero
         GTEST_ASSERT_EQ(bw_buffer[0] & 0x80, 0);