commit b72a5500aac7d1313e0304807ffc8e0509ae97b9
author Andy Hung <hunga@google.com> Mon Mar 27 15:53:06 2023 -0700
committer Andy Hung <hunga@google.com> Tue Apr 25 10:44:54 2023 -0700
tree 0a3a6fa8909e228cb009015e4b00b7c3ab249317
parent 580a0d5508adfff81334e30fed33889b99975eaf

AudioParameter: Document usage of exiting and closing.

Ensure only privileged access.
Log warning message on setParameters failure
to prevent entering standby silently.

Test: adb shell dumpsys media.audio_flinger
Bug: 73175392
Bug: 270480994
Bug: 279106598
Merged-In: Ie38ee93e98ff2164f9076b8b6b7831cc6b7a8896
Change-Id: Ie38ee93e98ff2164f9076b8b6b7831cc6b7a8896

services/audioflinger/Threads.cpp

diff --git a/services/audioflinger/Threads.cpp b/services/audioflinger/Threads.cpp
index 9e68cd3..02d058f 100644
--- a/services/audioflinger/Threads.cpp
+++ b/services/audioflinger/Threads.cpp
@@ -6056,12 +6056,12 @@
     if (status == NO_ERROR) {
         status = mOutput->stream->setParameters(keyValuePair);
         if (!mStandby && status == INVALID_OPERATION) {
+            ALOGW("%s: setParameters failed with keyValuePair %s, entering standby",
+                    __func__, keyValuePair.c_str());
             mOutput->standby();
-            if (!mStandby) {
-                mThreadMetrics.logEndInterval();
-                mThreadSnapshot.onEnd();
-                mStandby = true;
-            }
+            mThreadMetrics.logEndInterval();
+            mThreadSnapshot.onEnd();
+            mStandby = true;
             mBytesWritten = 0;
             status = mOutput->stream->setParameters(keyValuePair);
         }