commit e8b9239738aff472ffebb154190ae0abad12beb1
author ywan171 <yi.a.wang@intel.com> Thu May 08 10:59:09 2014 +0800
committer Patrick Tjin <pattjin@google.com> Mon Jul 21 21:51:53 2014 -0700
tree 77d1d2b0dd430d4c7b3ebab88637e9fd4f466363
parent c6b0b722e2caf081f2c2d3b91439036a738d7b60

libmix: when terminate VA, reset mSignalBufferPre[] even va is not initialized

BZ: 191657

when terminate VA, reset mSignalBufferPre[] even va is not initialized
or else after output reconfiguration, unreseted mSignalBufferPre[] will cause
buffer renderdone flag mismatch and buffer status failure.

Change-Id: I8cffbbfc104882900718f7f85555f6cd6fc5a6b2
Signed-off-by: ywan171 <yi.a.wang@intel.com>

videodecoder/VideoDecoderBase.cpp

diff --git a/videodecoder/VideoDecoderBase.cpp b/videodecoder/VideoDecoderBase.cpp
index a5b73f4..30b92af 100644
--- a/videodecoder/VideoDecoderBase.cpp
+++ b/videodecoder/VideoDecoderBase.cpp
@@ -978,6 +978,11 @@
 }
 
 Decode_Status VideoDecoderBase::terminateVA(void) {
+    mSignalBufferSize = 0;
+    for (int i = 0; i < MAX_GRAPHIC_BUFFER_NUM; i++) {
+         mSignalBufferPre[i] = NULL;
+    }
+
     if (mVAStarted == false) {
         // VA hasn't been started yet
         return DECODE_SUCCESS;
@@ -1044,10 +1049,6 @@
     mVAStarted = false;
     mInitialized = false;
     mErrReportEnabled = false;
-    mSignalBufferSize = 0;
-    for (int i = 0; i < MAX_GRAPHIC_BUFFER_NUM; i++) {
-         mSignalBufferPre[i] = NULL;
-    }
     return DECODE_SUCCESS;
 }