Commit ce521ee6 authored Jul 05, 2019 by Iris Chang Committed by Wonsik Kim Nov 27, 2019

Fix status error to avoid releasing component twice

When flush action is stuck, component will be released,
and state will be changed to RELEASED.
Then flush completed or returned error, and state will be
changed to FLUSHED.
When shutdown, release is called again, and exception happens.

Solution: Change state to FLUSHED only when it is FLUSHING

Bug: 136790747
Test: It is a seldom issue, and only happens when flush action
is stuck. We test this patch by modifying code to make flush
time more than 3s.
Change-Id: I74cb2599a9e029332d600469f38630651d9f43c0

parent 73b065fe

media/codec2/sfplugin/CCodec.cpp

+3 −1

Original line number	Diff line number	Diff line
		@@ -1524,8 +1524,10 @@ void CCodec::flush() {

		{
		Mutexed<State>::Locked state(mState);
		if (state->get() == FLUSHING) {
		state->set(FLUSHED);
		}
		}
		mCallback->onFlushCompleted();
		}