BufferPool: Avoid implementation-defined behavior
Shifting a signed 32-bit value by 31 bits is implementation-defined
behavior, so we change the value to unsigned.
Bug: 147147992
Test: TreeHugger
Change-Id: Iaefb85f77e5a9f8e7cd361a25a1c05fffc35ccb1
diff --git a/media/bufferpool/2.0/AccessorImpl.cpp b/media/bufferpool/2.0/AccessorImpl.cpp
index 51d9a69..6111fea 100644
--- a/media/bufferpool/2.0/AccessorImpl.cpp
+++ b/media/bufferpool/2.0/AccessorImpl.cpp
@@ -139,7 +139,7 @@
}
#ifdef __ANDROID_VNDK__
-static constexpr uint32_t kSeqIdVndkBit = 1 << 31;
+static constexpr uint32_t kSeqIdVndkBit = 1U << 31;
#else
static constexpr uint32_t kSeqIdVndkBit = 0;
#endif