Return NULL instead of result when unable to load bitmap.

There are still cases where 'bitmap' could be null.
We shouldn't return a result when that's the case.

Returnning null appears to be supported by Picasso.
(see Picasso.NetworkRequestHandler)

Fixes AntennaPod/AntennaPod#947

1 files changed, 6 insertions, 0 deletions

diff --git a/core/src/main/java/de/danoeh/antennapod/core/asynctask/PicassoProvider.java b/core/src/main/java/de/danoeh/antennapod/core/asynctask/PicassoProvider.java
index 09fe0d654..8e47a5b71 100644
--- a/core/src/main/java/de/danoeh/antennapod/core/asynctask/PicassoProvider.java
+++ b/core/src/main/java/de/danoeh/antennapod/core/asynctask/PicassoProvider.java
@@ -170,6 +170,12 @@ public class PicassoProvider {
                     bitmap = decodeStreamFromFile(data, fallback);
                 }
             }
+
+            if (bitmap == null) {
+                Log.e(TAG, "Could not load media");
+                return null;
+            }
+
             return new Result(bitmap, Picasso.LoadedFrom.DISK);
 
         }