commit e140b31fbde8b3e9da99de2c094b80e37e47444c
author mahongpe <hongpeng.ma@intel.com> Fri Nov 06 00:44:27 2015 +0800
committer Marco Nelissen <marcone@google.com> Fri Nov 06 14:00:59 2015 -0800
tree 4ee5893a0368901e9c93c142aed2cd35195c4d22
parent 3aa46b0bcd69b855f775a8b1f928776ca0fb35f6

fix random SEGV issue which is caused by wild pointer in libmix

When resolution changes, we should flush video video pipeline at
first, then free surface buffers.

Bug: 23160948

Change-Id: I84782d886d8a484c2131d0b3f529f82b9c6fd8c9
Signed-off-by: mahongpe <hongpeng.ma@intel.com>

videocodec/OMXVideoDecoderBase.cpp

diff --git a/videocodec/OMXVideoDecoderBase.cpp b/videocodec/OMXVideoDecoderBase.cpp
index 6270cea..975ac4c 100644
--- a/videocodec/OMXVideoDecoderBase.cpp
+++ b/videocodec/OMXVideoDecoderBase.cpp
@@ -734,9 +734,10 @@
         this->ports[INPORT_INDEX]->SetPortDefinition(&paramPortDefinitionInput, true);
         this->ports[OUTPORT_INDEX]->SetPortDefinition(&paramPortDefinitionOutput, true);
 
+        ProcessorFlush(INPORT_INDEX);
+
         mVideoDecoder->freeSurfaceBuffers();
 
-        ProcessorFlush(INPORT_INDEX);
         this->ports[OUTPORT_INDEX]->ReportPortSettingsChanged();
 
         mFormatChanged = false;