Fix the media filter buffer issue in the Tuner default implementation
The current implementation does not clear the previous memcopy before
copying the next round of data into the buffer. This CL fixes the issue.
Test: atest VtsHalTvTunerV1_0TargetTest
Bug: 159027928
Change-Id: I3399c69ead046630d9c22e8fbe1e816f6ea529f9
diff --git a/tv/tuner/1.1/default/Dvr.cpp b/tv/tuner/1.1/default/Dvr.cpp
index bf4c77e..3a4ef1b 100644
--- a/tv/tuner/1.1/default/Dvr.cpp
+++ b/tv/tuner/1.1/default/Dvr.cpp
@@ -395,6 +395,7 @@
mDemux->sendFrontendInputToRecord(frameData, pid, static_cast<uint64_t>(esMeta[i].pts));
}
startFilterDispatcher(isVirtualFrontend, isRecording);
+ frameData.clear();
}
return true;
diff --git a/tv/tuner/1.1/default/Filter.cpp b/tv/tuner/1.1/default/Filter.cpp
index 2d6214d..4d08afe 100644
--- a/tv/tuner/1.1/default/Filter.cpp
+++ b/tv/tuner/1.1/default/Filter.cpp
@@ -500,7 +500,12 @@
}
if (mPts) {
- return createMediaFilterEventWithIon(mFilterOutput);
+ Result result;
+ result = createMediaFilterEventWithIon(mFilterOutput);
+ if (result == Result::SUCCESS) {
+ mFilterOutput.clear();
+ }
+ return result;
}
for (int i = 0; i < mFilterOutput.size(); i += 188) {