commit 9680b52a7b55c5a70ddc585dc9dac93b40e6cf2c
author Weiyin Jiang <wjiang@codeaurora.org> Tue Apr 30 15:40:32 2019 +0800
committer Gerrit - the friendly Code Review server <code-review@localhost> Mon May 06 01:39:44 2019 -0700
tree 06ece411cb7e5116253d58cee6e9088f86fa2e70
parent 919f4e8dc35a57e26104ebb2426514aea1a81b9a

audiopolicy: fix null pointer being dereferenced in apm

Fix null pointer being dereferenced in apm.

Change-Id: Ibc2ec240c2da31a80ac087ed0f57661b10e97fb3
CRs-Fixed: 2444371

policy_hal/AudioPolicyManager.cpp

diff --git a/policy_hal/AudioPolicyManager.cpp b/policy_hal/AudioPolicyManager.cpp
index 99f1a24..2806284 100644
--- a/policy_hal/AudioPolicyManager.cpp
+++ b/policy_hal/AudioPolicyManager.cpp
@@ -1969,6 +1969,11 @@
 
     audio_io_handle_t input = inputDesc->mIoHandle;
     sp<RecordClientDescriptor> client = inputDesc->getClient(portId);
+    if (client == NULL) {
+        ALOGW("%s invalid client desc for %d", __FUNCTION__, portId);
+        return BAD_VALUE;
+    }
+
     if (client->active()) {
         ALOGW("%s input %d client %d already started", __FUNCTION__, input, client->portId());
         return INVALID_OPERATION;