From 3522b5a579090f09ec4d304ee4b8fca6c755d39c Mon Sep 17 00:00:00 2001 From: Andreas Huber Date: Fri, 22 Jan 2010 14:36:53 -0800 Subject: [PATCH] Make sure to release the renderer before shutting down the hardware codec, qcom's hardware decoder will otherwise crash. related-to-bug: 2387981 --- media/libstagefright/AwesomePlayer.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/media/libstagefright/AwesomePlayer.cpp b/media/libstagefright/AwesomePlayer.cpp index cc4511413434c..f6cd46ac87b53 100644 --- a/media/libstagefright/AwesomePlayer.cpp +++ b/media/libstagefright/AwesomePlayer.cpp @@ -218,6 +218,8 @@ void AwesomePlayer::reset() { void AwesomePlayer::reset_l() { cancelPlayerEvents(); + mVideoRenderer.clear(); + if (mLastVideoBuffer) { mLastVideoBuffer->release(); mLastVideoBuffer = NULL; @@ -243,8 +245,6 @@ void AwesomePlayer::reset_l() { delete mAudioPlayer; mAudioPlayer = NULL; - mVideoRenderer.clear(); - mDurationUs = -1; mFlags = 0; mVideoWidth = mVideoHeight = -1;