commit e69cada3e096a21b69768e921dd60241238c6b9a
author Nate Myren <ntmyren@google.com> Thu Dec 10 10:00:36 2020 -0800
committer Nate Myren <ntmyren@google.com> Mon Dec 14 13:01:52 2020 -0800
tree bc0d785b192aee94d63f96cf0dd6d4cdf516e1f7
parent 5dc9ad75a27c19ec5450c57ba08d7e04494b704b

attribute REMOTE_SUBMIX recording to RECORD_AUDIO_OUTPUT

REMOTE_SUBMIX audioRecords do not use the microphone, so attribute their
usage to RECORD_AUDIO_OUTPUT, instead of RECORD_AUDIO

Test: manual
Fixes: 173797503
Change-Id: Ibc5d09a6b2a90211781076f70516c03ed024cc42

services/audiopolicy/service/AudioPolicyInterfaceImpl.cpp

diff --git a/services/audiopolicy/service/AudioPolicyInterfaceImpl.cpp b/services/audiopolicy/service/AudioPolicyInterfaceImpl.cpp
index 0d604c7..b1eb0cb 100644
--- a/services/audiopolicy/service/AudioPolicyInterfaceImpl.cpp
+++ b/services/audiopolicy/service/AudioPolicyInterfaceImpl.cpp
@@ -574,7 +574,7 @@
 
     // check calling permissions
     if (!(startRecording(client->opPackageName, client->pid, client->uid,
-            client->attributes.source == AUDIO_SOURCE_HOTWORD)
+            client->attributes.source)
             || client->attributes.source == AUDIO_SOURCE_FM_TUNER)) {
         ALOGE("%s permission denied: recording not allowed for uid %d pid %d",
                 __func__, client->uid, client->pid);
@@ -663,7 +663,7 @@
         client->startTimeNs = 0;
         updateUidStates_l();
         finishRecording(client->opPackageName, client->uid,
-                        client->attributes.source == AUDIO_SOURCE_HOTWORD);
+                        client->attributes.source);
     }
 
     return status;
@@ -690,7 +690,7 @@
 
     // finish the recording app op
     finishRecording(client->opPackageName, client->uid,
-                    client->attributes.source == AUDIO_SOURCE_HOTWORD);
+                    client->attributes.source);
     AutoCallerClear acc;
     return mAudioPolicyManager->stopInput(portId);
 }