commit 47785a853ef0eb7aa54a99dd60b65f78b9605a60
author yidongh <yidongh@codeaurora.org> Mon May 08 19:29:29 2017 +0800
committer Gerrit - the friendly Code Review server <code-review@localhost> Mon May 08 22:59:41 2017 -0700
tree f575468c66f0675f07c939925736739e568dc034
parent 0d62af54e7c72b3212660a80925cf2ea6ef5efeb

hal: avoid dereferencing null pointer

Fix bug where null pointer may be dereferenced.

CRs-Fixed: 2043993
Change-Id: I5745a432155e858b4a680aca9363b8d76306f784

hal/audio_hw.c

diff --git a/hal/audio_hw.c b/hal/audio_hw.c
index cc150b2..4d476da 100644
--- a/hal/audio_hw.c
+++ b/hal/audio_hw.c
@@ -1522,6 +1522,7 @@
             }
         } else if (voice_extn_compress_voip_is_active(adev)) {
             bool out_snd_device_backend_match = true;
+            voip_usecase = get_usecase_from_list(adev, USECASE_COMPRESS_VOIP_CALL);
             if (usecase->stream.out != NULL) {
                 out_snd_device_backend_match = platform_check_backends_match(
                                                    voip_usecase->out_snd_device,
@@ -1529,7 +1530,6 @@
                                                        adev->platform,
                                                        usecase->stream.out));
             }
-            voip_usecase = get_usecase_from_list(adev, USECASE_COMPRESS_VOIP_CALL);
             if ((voip_usecase) && ((voip_usecase->devices & AUDIO_DEVICE_OUT_ALL_CODEC_BACKEND) &&
                 ((usecase->devices & AUDIO_DEVICE_OUT_ALL_CODEC_BACKEND) ||
                   ((usecase->devices & ~AUDIO_DEVICE_BIT_IN) & AUDIO_DEVICE_IN_ALL_CODEC_BACKEND)) &&