commit 7586e297d13f871fc4c71a7b40e2f932495ed14d
author Naresh Tanniru <ntanniru@codeaurora.org> Thu Apr 13 10:38:13 2017 +0530
committer Satya Krishna Pindiproli <satyak@codeaurora.org> Thu Jun 08 09:47:33 2017 +0530
tree 42bdf0e81580adb3b1f8c023da3c38518f35304b
parent b4288b8e009a6142e2fe71e917912deedde2b650

hal : Add proper check to fix NULL check error.

 -Add proper check to avoid dereferencing of
  pointer after null check.

Conflicts:
	hal/audio_extn/utils.c

Change-Id: I61e774b46dcca64e40fc0f5453abb673d8d0c09f

hal/audio_extn/utils.c

diff --git a/hal/audio_extn/utils.c b/hal/audio_extn/utils.c
index d9f4a0b..a2dff9f 100644
--- a/hal/audio_extn/utils.c
+++ b/hal/audio_extn/utils.c
@@ -1632,7 +1632,7 @@
                 ret = -EINVAL;
         }
     } else {
-        ALOGE("%s: Invalid usecase %d ",__func__, usecase->type);
+        ALOGE("%s: Invalid usecase",__func__);
         ret = -EINVAL;
     }
 
@@ -1768,12 +1768,7 @@
     struct stream_out *out = NULL;
     int ret = -EINVAL;
 
-    if (usecase == NULL) {
-        ALOGE("%s:: Invalid use case", __func__);
-        goto exit;
-    }
-
-    if (usecase->type != PCM_PLAYBACK) {
+    if (usecase == NULL || usecase->type != PCM_PLAYBACK) {
         ALOGE("%s:: Invalid use case", __func__);
         goto exit;
     }