fix #259 getExternalFilesDirs may be null if shared storage is not cu…

…rrently available (#260)
getsentry · Feb 4, 2020 · 291414a · 291414a
1 parent 416053c
commit 291414a
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/sentry-android-core/src/main/java/io/sentry/android/core/DefaultAndroidEventProcessor.java b/sentry-android-core/src/main/java/io/sentry/android/core/DefaultAndroidEventProcessor.java
@@ -658,6 +658,11 @@ private File getExternalStorageDep(File internalStorage) {
       String internalStoragePath =
           internalStorage != null ? internalStorage.getAbsolutePath() : null;
       for (File file : externalFilesDirs) {
+        // externalFilesDirs may contain null values :(
+        if (file == null) {
+          continue;
+        }
+
         // return the 1st file if you cannot compare with the internal one
         if (internalStoragePath == null || internalStoragePath.isEmpty()) {
           return file;