commit d98ea3b04b60d5b77b2c905ae051902c08e38863
author Linus Nilsson <lnilsson@google.com> Thu Oct 15 09:27:18 2020 -0700
committer Linus Nilsson <lnilsson@google.com> Thu Oct 15 09:27:18 2020 -0700
tree dad02388e7755a2e2c56ff2697622c0856e02912
parent 82a6a29675c8709f608cb12fb1cb11a28512eec0

Transcoder: Fix oob read in benchmark test

Fixes: 170961014
Test: Running benchmark under asan
Change-Id: I5eddcabb3a393ed6c646fb974034b3cb56e75dd1

media/libmediatranscoding/transcoder/benchmark/MediaTrackTranscoderBenchmark.cpp

diff --git a/media/libmediatranscoding/transcoder/benchmark/MediaTrackTranscoderBenchmark.cpp b/media/libmediatranscoding/transcoder/benchmark/MediaTrackTranscoderBenchmark.cpp
index c695ed9..aee0ed6 100644
--- a/media/libmediatranscoding/transcoder/benchmark/MediaTrackTranscoderBenchmark.cpp
+++ b/media/libmediatranscoding/transcoder/benchmark/MediaTrackTranscoderBenchmark.cpp
@@ -192,7 +192,7 @@
         auto& p = mSamples[mCurrentSampleIndex % mSamples.size()];
 
         if (bufferSize < p.second.size) return AMEDIA_ERROR_INVALID_PARAMETER;
-        memcpy(buffer, p.first.get(), bufferSize);
+        memcpy(buffer, p.first.get(), p.second.size);
 
         advanceTrack(trackIndex);
         return AMEDIA_OK;