commit 4069d5dcbe17ad70389730041b3f6503d3ae2697
author yuanjiahsu <yuanjiahsu@google.com> Mon Apr 19 07:54:27 2021 +0800
committer yuanjiahsu <yuanjiahsu@google.com> Mon Apr 19 16:03:07 2021 +0800
tree 5d024baa77e66e3aaead1153d1011be9ee9bed18
parent ff81743e3165f9a22a3858ea9f7d41851db5d358

Add nullptr check before printing found device type

Bug: 185106633
Test: manually
Change-Id: I5ed272cf29b207c2ccaaff468345dcf5c615b912

services/audiopolicy/managerdefault/AudioPolicyManager.cpp

diff --git a/services/audiopolicy/managerdefault/AudioPolicyManager.cpp b/services/audiopolicy/managerdefault/AudioPolicyManager.cpp
index dd44c54..ad359ec 100644
--- a/services/audiopolicy/managerdefault/AudioPolicyManager.cpp
+++ b/services/audiopolicy/managerdefault/AudioPolicyManager.cpp
@@ -2229,7 +2229,8 @@
             // Prevent from storing invalid requested device id in clients
             requestedDeviceId = AUDIO_PORT_HANDLE_NONE;
             device = mEngine->getInputDeviceForAttributes(attributes, uid, &policyMix);
-            ALOGV("%s found device type is 0x%X", __FUNCTION__, device->type());
+            ALOGV_IF(device != nullptr, "%s found device type is 0x%X",
+                __FUNCTION__, device->type());
         }
         if (device == nullptr) {
             ALOGW("getInputForAttr() could not find device for source %d", attributes.source);