commit d17c9125e42879694c97f9462eca0f5f0a60fd8f
author Sharad Sangle <assangle@codeaurora.org> Mon Mar 20 15:58:52 2017 +0530
committer Gerrit - the friendly Code Review server <code-review@localhost> Mon Apr 10 07:22:09 2017 -0700
tree dfde1d84b2a4bdb5486f153f95dc767807d8992c
parent 29b894e8c465802d2343f2e5956df47a90a56eb6

hal: In case of error in in_read don't return zero

When read fails HAL is returning bytes_read as zero, this
is causing recordThread forever looping in threadLoop, leading
ANR in cases like in-call record + SubSystemRestart.
Make sure HAL returns 'size', AudioFlinger is reading data of.

CRs-Fixed: 2021657
Change-Id: Ie4c07c6a888c1ec66ed032b0d2912633cf15368c

hal/audio_hw.c

diff --git a/hal/audio_hw.c b/hal/audio_hw.c
index 6644b48..1b0e1f1 100644
--- a/hal/audio_hw.c
+++ b/hal/audio_hw.c
@@ -3794,7 +3794,10 @@
             pthread_mutex_unlock(&adev->lock);
             in->standby = true;
         }
-        memset(buffer, 0, bytes);
+        if (!audio_extn_cin_attached_usecase(in->usecase)) {
+            bytes_read = bytes;
+            memset(buffer, 0, bytes);
+        }
         in_standby(&in->stream.common);
         ALOGV("%s: read failed status %d- sleeping for buffer duration", __func__, ret);
         usleep((uint64_t)bytes * 1000000 / audio_stream_in_frame_size(stream) /