commit c2afbdf2a57dd2e84ae2fe20e72c45c67b1aa4da
author Kevin Rocard <krocard@google.com> Thu Jan 31 18:18:06 2019 -0800
committer Kevin Rocard <krocard@google.com> Wed Feb 06 16:13:13 2019 -0800
tree bad704e7e1461de7372754856e84042428f2fd5b
parent c43ea147033817dc46b76d8993a84f0e5ee4e4e0

Dynamic audio policy should reject non pcm

Test: adb shell audiorecorder --target /data/file.raw
Bug: 111453086
Change-Id: I4691b19b1fa2f3b8e8e5045c9ea5be0a5c9effb6
Signed-off-by: Kevin Rocard <krocard@google.com>

services/audiopolicy/managerdefault/AudioPolicyManager.cpp

diff --git a/services/audiopolicy/managerdefault/AudioPolicyManager.cpp b/services/audiopolicy/managerdefault/AudioPolicyManager.cpp
index 02b85b4..391e38d 100644
--- a/services/audiopolicy/managerdefault/AudioPolicyManager.cpp
+++ b/services/audiopolicy/managerdefault/AudioPolicyManager.cpp
@@ -951,8 +951,8 @@
 
     // FIXME: in case of RENDER policy, the output capabilities should be checked
     if ((usePrimaryOutputFromPolicyMixes || !secondaryDescs->empty())
-        && !audio_has_proportional_frames(config->format)) {
-        ALOGW("%s: audio loopback only supports proportional frames", __func__);
+        && !audio_is_linear_pcm(config->format)) {
+        ALOGD("%s: rejecting request as dynamic audio policy only support pcm", __func__);
         return BAD_VALUE;
     }
     if (usePrimaryOutputFromPolicyMixes) {