summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Robert Shih <robertshih@google.com> 2018-11-08 11:00:05 -0800
committer Robert Shih <robertshih@google.com> 2018-11-13 20:22:54 -0800
commitc88f57609f7f0ab5f82dd65e84429dd29dff26c3 (patch)
tree1c1a82feb34f04024b1d8f5a4fa754ce3246f066
parent2823d3abdd51e5bac9021dcc176ebd6abacc77e5 (diff)
MediaPlayer2Impl: avoid NPE when accessing mHandlerThread
Bug: 109928575 Change-Id: I43d4ea528f702482ac197391b5bf236ff3794072
-rw-r--r--media/java/android/media/MediaPlayer2Impl.java6
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");
}