commit 599ec462d94136949e0c90419ea99b0d9ba83eac
author Svetoslav Ganov <svetoslavganov@google.com> Thu Mar 01 22:19:55 2018 +0000
committer Svet Ganov <svetoslavganov@google.com> Thu Mar 01 16:10:34 2018 -0800
tree 0774e6abd59eec5530f78d6788a8da27217bac63
parent 9b887324b8c3f86f012915a6f637081b158a221b

Revert "Use start/finish app op API for mic use"

This reverts commit 55bb217cd1fea3e791d0d7cf73e09e27fed8c36e.

Reason for revert: Causes a regression

bug:74061812

Change-Id: I119937cfce79ba824185634f2b81dad1606a875d

services/audiopolicy/service/AudioPolicyInterfaceImpl.cpp

diff --git a/services/audiopolicy/service/AudioPolicyInterfaceImpl.cpp b/services/audiopolicy/service/AudioPolicyInterfaceImpl.cpp
index d04b21e..306de3f 100644
--- a/services/audiopolicy/service/AudioPolicyInterfaceImpl.cpp
+++ b/services/audiopolicy/service/AudioPolicyInterfaceImpl.cpp
@@ -412,7 +412,7 @@
     }
 
     // check calling permissions
-    if (!startRecording(client->opPackageName, client->pid, client->uid)) {
+    if (!recordingAllowed(client->opPackageName, client->pid, client->uid)) {
         ALOGE("%s permission denied: recording not allowed for uid %d pid %d",
                 __func__, client->uid, client->pid);
         return PERMISSION_DENIED;
@@ -457,9 +457,6 @@
     }
     sp<AudioRecordClient> client = mAudioRecordClients.valueAt(index);
 
-    // finish the recording app op
-    finishRecording(client->opPackageName, client->uid);
-
     return mAudioPolicyManager->stopInput(client->input, client->session);
 }