commit 50ebcbf8f9c56c494de23e2db5214b666bbc1320
author Anish Kumar <kanish@codeaurora.org> Tue Dec 09 04:01:39 2014 +0530
committer Anish Kumar <kanish@codeaurora.org> Wed Dec 10 06:44:54 2014 +0530
tree 8b725b612ea4f43ff8cea45b69bc82e79987e1b8
parent 098dcbdd502059bd7443b15efb8685d634a86255

hal: fix null pointer de-reference

When speaker protection is turned on input stream can be
NULL. Add a check to fix de-referencing NULL pointer.

Change-Id: I75647d9360a559b5255c29d0baca264b1274eba2

hal/audio_hw.c

diff --git a/hal/audio_hw.c b/hal/audio_hw.c
index 26088eb..28ab0d6 100644
--- a/hal/audio_hw.c
+++ b/hal/audio_hw.c
@@ -1995,7 +1995,7 @@
            (out->config.rate);
     }
 
-    ALOGV("%s: Latency %d", latency);
+    ALOGV("%s: Latency %d", __func__, latency);
     return latency;
 }
 

hal/voice.c

diff --git a/hal/voice.c b/hal/voice.c
index cf5ec65..5fb0956 100644
--- a/hal/voice.c
+++ b/hal/voice.c
@@ -222,6 +222,11 @@
 {
     bool in_call_rec = false;
 
+    if (!in) {
+       ALOGE("%s: input stream is NULL", __func__);
+       return in_call_rec;
+    }
+
     if(in->source == AUDIO_SOURCE_VOICE_DOWNLINK ||
        in->source == AUDIO_SOURCE_VOICE_UPLINK ||
        in->source == AUDIO_SOURCE_VOICE_CALL) {