commit c045a03e3631317a96e15fd368d65294006c9de5
author Shunkai Yao <yaoshunkai@google.com> Tue Dec 27 21:33:17 2022 +0000
committer Shunkai Yao <yaoshunkai@google.com> Wed Dec 28 01:33:08 2022 +0000
tree a7ec19eb9fb4c1c9ddb7676e92174fc3f94ae740
parent 351317028e1cbc07e9a3a746c1acbb74d2769b70

Effect AIDL Check visualizer latency parameter

Add default value for class members.
Update mutex locking for some members.

Bug: 258124419
Test: atest VtsHalVisualizerTargetTest
Change-Id: I98c9b7a488e1d6d910428eae7e9f4cb011deb13e

media/libeffects/visualizer/aidl/Visualizer.cpp

diff --git a/media/libeffects/visualizer/aidl/Visualizer.cpp b/media/libeffects/visualizer/aidl/Visualizer.cpp
index e21aa01..28a7287 100644
--- a/media/libeffects/visualizer/aidl/Visualizer.cpp
+++ b/media/libeffects/visualizer/aidl/Visualizer.cpp
@@ -101,7 +101,10 @@
     auto tag = param.getTag();
     switch (tag) {
         case Visualizer::SetOnlyParameters::latencyMs: {
-            mContext->setDownstreamLatency(param.get<Visualizer::SetOnlyParameters::latencyMs>());
+            RETURN_IF(mContext->setDownstreamLatency(
+                              param.get<Visualizer::SetOnlyParameters::latencyMs>()) !=
+                              RetCode::SUCCESS,
+                      EX_ILLEGAL_ARGUMENT, "setLatencyFailed");
             break;
         }
         default: {