commit 2407ce3db8f7d90aa267b05ad1873eb470e3062c
author Eric Laurent <elaurent@google.com> Mon Apr 26 14:56:03 2021 +0200
committer Eric Laurent <elaurent@google.com> Wed Apr 28 14:15:13 2021 +0200
tree 5ee37b17ef9c772ae05c1ec5e5c6073101be3f8e
parent ab113ac81467f425992793f933a8d9a54e3deee3

audio flinger: fix wraparound issues in AudioRecord shared audi history implementation

Fix issues with the implementation of the shared audio history record start
point in RecordThread input resampling buffer when indexes wrapround.

Bug: 185972521
Test: atest AudioRecordSharedAudioTest
Change-Id: Idcb664f1253b5a488991b39647c13087fc46b0c4

services/audioflinger/Tracks.cpp

diff --git a/services/audioflinger/Tracks.cpp b/services/audioflinger/Tracks.cpp
index 6549236..3e04804 100644
--- a/services/audioflinger/Tracks.cpp
+++ b/services/audioflinger/Tracks.cpp
@@ -2378,7 +2378,7 @@
             audio_input_flags_t flags,
             track_type type,
             audio_port_handle_t portId,
-            int64_t startTimeMs)
+            int32_t startFrames)
     :   TrackBase(thread, client, attr, sampleRate, format,
                   channelMask, frameCount, buffer, bufferSize, sessionId,
                   creatorPid,
@@ -2396,7 +2396,7 @@
         mFlags(flags),
         mSilenced(false),
         mOpRecordAudioMonitor(OpRecordAudioMonitor::createIfNeeded(identity, attr)),
-        mStartTimeMs(startTimeMs)
+        mStartFrames(startFrames)
 {
     if (mCblk == NULL) {
         return;