commit 81ad5ecd211d0f30225714ae8ee61c4d76dbcf56
author Phil Burk <philburk@google.com> Fri Sep 01 10:45:41 2017 -0700
committer Phil Burk <philburk@google.com> Tue Sep 05 09:08:23 2017 -0700
tree 0160fa4543c2a83f34863a373d0854119334dd76
parent 3b224af571e8e70ef643a36e144e7427857d271d

audioflinger: pass return code up from startClient

Was ignoring failed start.

Bug: 65262790
Test: see bug
Change-Id: I0b1d19adfd50d2b00d86bb82f278b720fcbacdb1

services/audioflinger/Threads.cpp

diff --git a/services/audioflinger/Threads.cpp b/services/audioflinger/Threads.cpp
index d951e93..297608c 100644
--- a/services/audioflinger/Threads.cpp
+++ b/services/audioflinger/Threads.cpp
@@ -7633,6 +7633,10 @@
         return NO_ERROR;
     }
 
+    if (!isOutput() && !recordingAllowed(client.packageName, client.clientPid, client.clientUid)) {
+        return PERMISSION_DENIED;
+    }
+
     audio_port_handle_t portId = AUDIO_PORT_HANDLE_NONE;
 
     audio_io_handle_t io = mId;