展锐平台NewGallery2内存泄露分析

客户在使用展锐平台在文件管理器中打开一个视频循环播放，长时间发现播放自动停止，无法继续播放

通过排查log发现出现如下问题

08-08 19:34:35.881 1941 1941 E JavaBinder: !!! FAILED BINDER TRANSACTION !!! (parcel size = 232)
08-08 19:34:35.920 1941 1941 W MoviePlayerVideoView: Unable to open the videocontent://com.android.providers.media.documents/document/video%3A13
08-08 19:34:35.920 1941 1941 W MoviePlayerVideoView: DeadSystemException: The system died; earlier logs will point to the root cause

通过定位代码发现未出现明显错误，网上对JavaBinder: !!! FAILED BINDER TRANSACTION !!! 错误的大部分原因都是intent传递的数据过大导致。

但是通过本地代码查看未出现intent相关的操作，但是仍然出现了FAILED BINDER TRANSACTION !，那么就要在binder失败入手，考虑到此问题是长时间运行后必现的问题，那么

就要往内存泄露的方向去考虑了，一次小的binder传输可能不会有啥问题，但是如果内存泄露积累多了，是不是就会导致 JavaBinder: !!! FAILED BINDER TRANSACTION !!! (parcel size = 232)

顺着这个思路我们去排查一下内存泄露，通过log来定位 W MoviePlayerVideoView: Unable to open the videocontent://com.android.providers.media.documents/document/video%3A13

二分法加log判断具体行数，最终定位到

MoviePlayerVideoView.java里的mMediaPlayer.setDataSource(mContext, mUri, mHeaders);

这一行，通过查看MediaPlayer的代码可以发现setDataSource的时候确实去跟服务端进行了通讯，

前面的

mMediaPlayer.setOnPreparedListener(mPreparedListener);
mMediaPlayer.setOnVideoSizeChangedListener(mSizeChangedListener);
mMediaPlayer.setOnSeekCompleteListener(mSeekCompleteListener); // SPRD: add
mMediaPlayer.setOnCompletionListener(mCompletionListener);
mMediaPlayer.setOnErrorListener(mErrorListener);
mMediaPlayer.setOnInfoListener(mInfoListener);// MERGETEMP
mMediaPlayer.setOnBufferingUpdateListener(mBufferingUpdateListener);

这些操作，包括
mMediaPlayer = new MediaPlayer();

的时候都没有去binder服务端。

我们把这段代码整个贴出来，

private void openVideo() {
if (mUri == null || mSurfaceHolder == null) {
/** SPRD:not ready for playback just yet, will try again later */
return;
}
Log.i(TAG, "openVideo()");
/* SPRD:Delete for bug609816 The wrong logic for audiofocus @{
AudioManager am = (AudioManager) mContext.getSystemService(Context.AUDIO_SERVICE);
am.requestAudioFocus(null, AudioManager.STREAM_MUSIC,
AudioManager.AUDIOFOCUS_GAIN_TRANSIENT);*/
/* Bug609816 end @} */
/**
* SPRD:we shouldn't clear the target state, because somebody might have called start()
* previously
*/
release(false);

try {
mMediaPlayer = new MediaPlayer();
// TODO: create SubtitleController in MediaPlayer, but we need
/** SPRD: a context for the subtitle renderers */
final Context context = getContext();
StandardFrameworks.getInstances().setVideoViewRendererAndSubtitleAnchor(context, mMediaPlayer, this);

if (mAudioSession != 0) {
mMediaPlayer.setAudioSessionId(mAudioSession);
} else {
mAudioSession = mMediaPlayer.getAudioSessionId();
}
mMediaPlayer.setOnPreparedListener(mPreparedListener);
mMediaPlayer.setOnVideoSizeChangedListener(mSizeChangedListener);
mMediaPlayer.setOnSeekCompleteListener(mSeekCompleteListener); // SPRD: add
mMediaPlayer.setOnCompletionListener(mCompletionListener);
mMediaPlayer.setOnErrorListener(mErrorListener);
mMediaPlayer.setOnInfoListener(mInfoListener);// MERGETEMP
mMediaPlayer.setOnBufferingUpdateListener(mBufferingUpdateListener);
mCurrentBufferPercentage = 0;
mMediaPlayer.setDataSource(mContext, mUri, mHeaders);
mMediaPlayer.setDisplay(mSurfaceHolder);
mMediaPlayer.setAudioStreamType(AudioManageer.STREAM_MUSIC);
mMediaPlayer.setScreenOnWhilePlaying(true);

mMediaPlayer.prepareAsync();

/* SPRD:bug474646 Add Drm feature modify by old bug506989
* we can not consume when video trun to background. 355
@{ */
Log.d(TAG, "if need consume " + mNeedConsumeDrmRight);
// SPRD: Bug568552, temp modify for AndroidN porting @{
// mMediaPlayer.setNeedToConsume(mNeedConsumeDrmRight);
if (mNeedConsumeDrmRight) {
Log.i(TAG, "consumeDrmRights");
StandardFrameworks.getInstances().consumeDrmRights(mMediaPlayer);
}
// @}
/** @} */
for (Pair pending : mPendingSubtitleTracks) {
addSubtitleSource(pending.first, pending.second);
}

/**
* SPRD:remove for 5.0 mMediaPlayer.setLastInterruptPosition(mInterruptPosition); //
* SPRD: add we don't set the target state here either, but preserve the target state
* that was there before.
*/
mCurrentState = STATE_PREPARING;
attachMediaController();
} catch (IOException ex) {
Log.w(TAG, "Unable to open content: " + mUri, ex);
mCurrentState = STATE_ERROR;
mTargetState = STATE_ERROR;
mErrorListener.onError(mMediaPlayer, MediaPlayer.MEDIA_ERROR_UNKNOWN, 0);
return;
} catch (IllegalArgumentException ex) {
Log.w(TAG, "Unable to open content: " + mUri, ex);
mCurrentState = STATE_ERROR;
mTargetState = STATE_ERROR;
mErrorListener.onError(mMediaPlayer, MediaPlayer.MEDIA_ERROR_UNKNOWN, 0);
return;

} catch (IllegalStateException ex) {
Log.w(TAG, "Unable to open" + mUri, ex);
mCurrentState = STATE_ERROR;
mTargetState = STATE_ERROR;
mErrorListener.onError(mMediaPlayer, MediaPlayer.MEDIA_ERROR_UNKNOWN, 0);
//add for bug529336 java.lang.NullPointerException
} catch (Exception ex) {
Log.w(TAG, "Unable to open the video" + mUri, ex);
mCurrentState = STATE_ERROR;
mTargetState = STATE_ERROR;
mErrorListener.onError(mMediaPlayer, MediaPlayer.MEDIA_ERROR_UNKNOWN, 0);
} finally {
mPendingSubtitleTracks.clear();
}
}