commit b0f5bfd12c03317d179e1cec64149f3216756af5
author juyuchen <juyuchen@google.com> Wed Dec 12 17:58:09 2018 +0800
committer Juyu Chen <juyuchen@google.com> Wed Dec 12 10:08:00 2018 +0000
tree 87e7b3b089f2683932e718b80061555248939714
parent f853dfd507f9df0c69369e984a7383accc869e7f

audio: fix potential messy offload state.

Bug: 119035383
Test: manual audio test

Change-Id: Ie87e1fa330bb7904fde2c429db5392ab24001510
Signed-off-by: juyuchen <juyuchen@google.com>

hal/audio_hw.c

diff --git a/hal/audio_hw.c b/hal/audio_hw.c
index 3312cbe..7661e58 100644
--- a/hal/audio_hw.c
+++ b/hal/audio_hw.c
@@ -1743,15 +1743,15 @@
     struct stream_out *out = (struct stream_out *) context;
     struct listnode *item;
 
-    out->offload_state = OFFLOAD_STATE_IDLE;
-    out->playback_started = 0;
-
     setpriority(PRIO_PROCESS, 0, ANDROID_PRIORITY_AUDIO);
     set_sched_policy(0, SP_FOREGROUND);
     prctl(PR_SET_NAME, (unsigned long)"Offload Callback", 0, 0, 0);
 
     ALOGV("%s", __func__);
+
     lock_output_stream(out);
+    out->offload_state = OFFLOAD_STATE_IDLE;
+    out->playback_started = 0;
     for (;;) {
         struct offload_cmd *cmd = NULL;
         stream_callback_event_t event;