diff options
| author | 2018-11-08 11:00:05 -0800 | |
|---|---|---|
| committer | 2018-11-13 20:22:54 -0800 | |
| commit | c88f57609f7f0ab5f82dd65e84429dd29dff26c3 (patch) | |
| tree | 1c1a82feb34f04024b1d8f5a4fa754ce3246f066 | |
| parent | 2823d3abdd51e5bac9021dcc176ebd6abacc77e5 (diff) | |
MediaPlayer2Impl: avoid NPE when accessing mHandlerThread
Bug: 109928575
Change-Id: I43d4ea528f702482ac197391b5bf236ff3794072
| -rw-r--r-- | media/java/android/media/MediaPlayer2Impl.java | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/media/java/android/media/MediaPlayer2Impl.java b/media/java/android/media/MediaPlayer2Impl.java index 4ac0188581e8..9b97b1060c8a 100644 --- a/media/java/android/media/MediaPlayer2Impl.java +++ b/media/java/android/media/MediaPlayer2Impl.java @@ -700,7 +700,8 @@ public final class MediaPlayer2Impl extends MediaPlayer2 { // return true if there is a next data source, false otherwise. // This function should be always called on |mHandlerThread|. private boolean prepareNextDataSource() { - if (Looper.myLooper() != mHandlerThread.getLooper()) { + HandlerThread handlerThread = mHandlerThread; + if (handlerThread != null && Looper.myLooper() != handlerThread.getLooper()) { Log.e(TAG, "prepareNextDataSource: called on wrong looper"); } @@ -736,7 +737,8 @@ public final class MediaPlayer2Impl extends MediaPlayer2 { // This function should be always called on |mHandlerThread|. private void playNextDataSource() { - if (Looper.myLooper() != mHandlerThread.getLooper()) { + HandlerThread handlerThread = mHandlerThread; + if (handlerThread != null && Looper.myLooper() != handlerThread.getLooper()) { Log.e(TAG, "playNextDataSource: called on wrong looper"); } |