Fix random NullPointerException when adding video player view

add TeamNewPipe/NewPipe#8661 to legacy flora Co-Authored-By: Stypox <stypox@pm.me>
ShareASmile · Jun 22, 2024 · 049b468 · 049b468
1 parent b776584
commit 049b468
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/app/src/main/java/org/schabi/newpipelegacy/fragments/detail/VideoDetailFragment.java b/app/src/main/java/org/schabi/newpipelegacy/fragments/detail/VideoDetailFragment.java
@@ -1323,9 +1323,11 @@ private void addVideoPlayerView() {
         // Prevent from re-adding a view multiple times
         new Handler(Looper.getMainLooper()).post(() ->
                 player.UIs().get(MainPlayerUi.class).ifPresent(playerUi -> {
-                    playerUi.removeViewFromParent();
-                    binding.playerPlaceholder.addView(playerUi.getBinding().getRoot());
-                    playerUi.setupVideoSurfaceIfNeeded();
+                    if (binding != null) {
+                        playerUi.removeViewFromParent();
+                        binding.playerPlaceholder.addView(playerUi.getBinding().getRoot());
+                        playerUi.setupVideoSurfaceIfNeeded();
+                    }
                 }));
     }