commit ad936af3dbe4db69a74472874ada08739b564ac3
author Songyue Han <songyueh@google.com> Mon Oct 14 23:53:58 2024 +0000
committer Songyue Han <songyueh@google.com> Thu Oct 24 22:06:15 2024 +0000
tree 873c1c069120f4f426ee74ca770ba84182b0ff98
parent 84ad37f5eeb194caf8f76777f81083e033632876

CodecCapabilities: Restrict bitRate from int to int32_t.

Prevent potential mul overflow issue.

Test: MediaCodecListTest
Bug: 306023029
Change-Id: Id4a0fc1473d6aa1b8ffdb453180fb0fc7de92d40

media/libmedia/tests/codeccapabilities/CodecCapabilitiesTest.cpp

diff --git a/media/libmedia/tests/codeccapabilities/CodecCapabilitiesTest.cpp b/media/libmedia/tests/codeccapabilities/CodecCapabilitiesTest.cpp
index 89c9739..02e43a4 100644
--- a/media/libmedia/tests/codeccapabilities/CodecCapabilitiesTest.cpp
+++ b/media/libmedia/tests/codeccapabilities/CodecCapabilitiesTest.cpp
@@ -64,7 +64,7 @@
 };
 
 TEST_F(AudioCapsAacTest, AudioCaps_Aac_Bitrate) {
-    const Range<int>& bitrateRange = audioCaps->getBitrateRange();
+    const Range<int32_t>& bitrateRange = audioCaps->getBitrateRange();
     EXPECT_EQ(bitrateRange.lower(), 8000) << "bitrate range1 does not match. lower: "
             << bitrateRange.lower();
     EXPECT_EQ(bitrateRange.upper(), 510000) << "bitrate range1 does not match. upper: "
@@ -114,7 +114,7 @@
 };
 
 TEST_F(AudioCapsRawTest, AudioCaps_Raw_Bitrate) {
-    const Range<int>& bitrateRange = audioCaps->getBitrateRange();
+    const Range<int32_t>& bitrateRange = audioCaps->getBitrateRange();
     EXPECT_EQ(bitrateRange.lower(), 1);
     EXPECT_EQ(bitrateRange.upper(), 10000000);
 }