commit ff56ff3557aa5ea22d5d2f68b436fd32d786920d
author Weiyin Jiang <wjiang@codeaurora.org> Fri May 08 14:32:21 2020 +0800
committer Weiyin Jiang <wjiang@codeaurora.org> Fri May 08 15:01:29 2020 +0800
tree bccc1c3badb4456b1ceb1fae8555e973e4beb24d
parent 40f2b321e8963736197b34cb7d10f78f53adde01

hal: add usecase type check

Add usecase type check to avoid crash.

CRs-Fixed: 2681468
Change-Id: I4927d4ff54350a88cfa0c17255331797230387a2

hal/audio_hw.c

diff --git a/hal/audio_hw.c b/hal/audio_hw.c
index 7e08c5c..1c13da0 100644
--- a/hal/audio_hw.c
+++ b/hal/audio_hw.c
@@ -8751,8 +8751,10 @@
         struct listnode *node;
         list_for_each(node, &adev->usecase_list) {
             usecase = node_to_item(node, struct audio_usecase, list);
-            if (usecase->stream.out && (usecase->type == PCM_PLAYBACK) &&
-                is_a2dp_out_device_type(&usecase->device_list)) {
+            if ((usecase->stream.out == NULL) || (usecase->type != PCM_PLAYBACK))
+                continue;
+
+            if (is_a2dp_out_device_type(&usecase->device_list)) {
                 ALOGD("reconfigure a2dp... forcing device switch");
                 pthread_mutex_unlock(&adev->lock);
                 lock_output_stream(usecase->stream.out);