Fix java.lang.NullPointerException in checkYoutubeOverride (#1437)

MozillaReality · Jul 22, 2019 · e5a2d5a · e5a2d5a
1 parent c81f61c
commit e5a2d5a
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/app/src/common/shared/org/mozilla/vrbrowser/browser/SessionStore.java b/app/src/common/shared/org/mozilla/vrbrowser/browser/SessionStore.java
@@ -1113,14 +1113,20 @@ else if (aRequest.triggerUri != null)
     private String checkYoutubeOverride(String aUri) {
         try {
             Uri uri = Uri.parse(aUri);
+            if (uri.getHost() == null) {
+                return null;
+            }
             String hostLower = uri.getHost().toLowerCase();
             if (!hostLower.endsWith(".youtube.com") && !hostLower.endsWith(".youtube-nocookie.com")) {
                 return null;
             }
 
             Uri.Builder uriBuilder = uri.buildUpon();
-            Boolean updateUri = false;
+            boolean updateUri = false;
 
+            if (uri.getScheme() == null) {
+                return null;
+            }
             if (!uri.getScheme().equalsIgnoreCase("https")) {
                 uriBuilder.scheme("https");
                 updateUri = true;