From a279310f9249c552d634d82f2452024d4201c15e Mon Sep 17 00:00:00 2001 From: Thomas Leplus Date: Thu, 16 Sep 2021 23:40:11 -0700 Subject: [PATCH] Avoid NPE when mediaPlayerDelegate is null If you call mediaPlayerDelegate.videoInfo before checking that mediaPlayerDelegate is not null, you'll get a NullPointerException. --- .../main/java/com/youku/player/plugin/PluginImageAD.java | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/library_youku/src/main/java/com/youku/player/plugin/PluginImageAD.java b/library_youku/src/main/java/com/youku/player/plugin/PluginImageAD.java index 8cf64d2..02c3e3c 100644 --- a/library_youku/src/main/java/com/youku/player/plugin/PluginImageAD.java +++ b/library_youku/src/main/java/com/youku/player/plugin/PluginImageAD.java @@ -663,9 +663,8 @@ public void onClick(View v) { } else { adImageView.setOnClickListener(null); } - if (StaticsUtil.PLAY_TYPE_LOCAL.equals(mediaPlayerDelegate.videoInfo - .getPlayType()) - && mediaPlayerDelegate != null + if (mediaPlayerDelegate != null + && StaticsUtil.PLAY_TYPE_LOCAL.equals(mediaPlayerDelegate.videoInfo.getPlayType()) && mediaPlayerDelegate.pluginManager != null) { mediaPlayerDelegate.pluginManager.onVideoInfoGetted(); mediaPlayerDelegate.pluginManager.onChangeVideo(); @@ -1701,4 +1700,4 @@ private boolean isLand() { } return false; } -} \ No newline at end of file +}