commit d0e64b139cb2b3390078f48b866a7c036e685d9b
author Shunkai Yao <yaoshunkai@google.com> Thu Jan 25 00:44:33 2024 +0000
committer Shunkai Yao <yaoshunkai@google.com> Sat Jan 27 00:13:23 2024 +0000
tree da483bd5949b4aa2599e0452c42a07f8d1b3925a
parent 76dbfecab6b4cb9be97385afdf50e50987709073

Check work buffer size before effect process

Bug: 321543210
Test: atest VtsHalAudioEffectTargetTest
Change-Id: I227197f0107f66866d099fb4d160bdb46a65b7df

media/libeffects/downmix/aidl/EffectDownmix.cpp

diff --git a/media/libeffects/downmix/aidl/EffectDownmix.cpp b/media/libeffects/downmix/aidl/EffectDownmix.cpp
index c82c23b..46156ce 100644
--- a/media/libeffects/downmix/aidl/EffectDownmix.cpp
+++ b/media/libeffects/downmix/aidl/EffectDownmix.cpp
@@ -195,6 +195,8 @@
         const auto availableToWrite = outputMQ->availableToWrite() *
                                       mImplContext->getInputFrameSize() /
                                       mImplContext->getOutputFrameSize();
+        assert(mImplContext->getWorkBufferSize() >=
+               std::max(availableToRead(), availableToWrite));
         auto processSamples = std::min(availableToRead, availableToWrite);
         if (processSamples) {
             inputMQ->read(buffer, processSamples);