commit 744be48d57da2c39f1479bc5a5ad0b7589814704
author Nadav Bar <nadavbar@google.com> Tue May 08 13:26:21 2018 +0300
committer Eric Laurent <elaurent@google.com> Tue May 08 16:00:00 2018 +0000
tree 7c3130520c6336b859837f6eefe02fe85d5fafd5
parent 84e84a5dd3b6d9ee4cf032e8d6206c70969b7107

Check voice call recording permission according to source.

Added a check for CAPTURE_AUDIO_OUTPUT according to the audio source, in
addition to the already existing check according to the device type.

Test: Tested manually.
Bug: 77228430.

Change-Id: I84bbe41c9abcdbbd25354b2c160ae47c9c959de4

services/audiopolicy/service/AudioPolicyInterfaceImpl.cpp

diff --git a/services/audiopolicy/service/AudioPolicyInterfaceImpl.cpp b/services/audiopolicy/service/AudioPolicyInterfaceImpl.cpp
index 008d655..364eacd 100644
--- a/services/audiopolicy/service/AudioPolicyInterfaceImpl.cpp
+++ b/services/audiopolicy/service/AudioPolicyInterfaceImpl.cpp
@@ -340,6 +340,13 @@
         return PERMISSION_DENIED;
     }
 
+    if ((attr->source == AUDIO_SOURCE_VOICE_UPLINK ||
+        attr->source == AUDIO_SOURCE_VOICE_DOWNLINK ||
+        attr->source == AUDIO_SOURCE_VOICE_CALL) &&
+        !captureAudioOutputAllowed(pid, uid)) {
+        return PERMISSION_DENIED;
+    }
+
     if ((attr->source == AUDIO_SOURCE_HOTWORD) && !captureHotwordAllowed(pid, uid)) {
         return BAD_VALUE;
     }