diff options
| author | Weiyin Jiang <wjiang@codeaurora.org> | 2019-04-30 15:40:32 +0800 |
|---|---|---|
| committer | Gerrit - the friendly Code Review server <code-review@localhost> | 2019-05-06 01:39:44 -0700 |
| commit | 9680b52a7b55c5a70ddc585dc9dac93b40e6cf2c (patch) | |
| tree | 06ece411cb7e5116253d58cee6e9088f86fa2e70 | |
| parent | 919f4e8dc35a57e26104ebb2426514aea1a81b9a (diff) | |
| download | android_vendor_qcom_opensource_audio-9680b52a7b55c5a70ddc585dc9dac93b40e6cf2c.tar.gz android_vendor_qcom_opensource_audio-9680b52a7b55c5a70ddc585dc9dac93b40e6cf2c.tar.bz2 android_vendor_qcom_opensource_audio-9680b52a7b55c5a70ddc585dc9dac93b40e6cf2c.zip | |
audiopolicy: fix null pointer being dereferenced in apm
Fix null pointer being dereferenced in apm.
Change-Id: Ibc2ec240c2da31a80ac087ed0f57661b10e97fb3
CRs-Fixed: 2444371
| -rw-r--r-- | policy_hal/AudioPolicyManager.cpp | 5 |
1 files changed, 5 insertions, 0 deletions
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 @@ status_t AudioPolicyManagerCustom::startInput(audio_port_handle_t portId) 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; |