From fb2608277adcc535409220df8c93eec724ecd17f Mon Sep 17 00:00:00 2001 From: Adrian Ulrich Date: Fri, 14 Aug 2015 10:05:18 +0200 Subject: [PATCH] never return coverart for unindexed songs (eg: missing album key!) --- src/ch/blinkenlights/android/vanilla/CoverCache.java | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/ch/blinkenlights/android/vanilla/CoverCache.java b/src/ch/blinkenlights/android/vanilla/CoverCache.java index 9d35be89..4776b3a4 100644 --- a/src/ch/blinkenlights/android/vanilla/CoverCache.java +++ b/src/ch/blinkenlights/android/vanilla/CoverCache.java @@ -198,6 +198,12 @@ public class CoverCache { * @param maxPxCount the maximum amount of pixels to return (30*30 = 900) */ public Bitmap createBitmap(Song song, long maxPxCount) { + + if (song.id < 0) { + // Unindexed song: return early + return null; + } + try { InputStream inputStream = null; InputStream sampleInputStream = null; // same as inputStream but used for getSampleSize @@ -224,7 +230,7 @@ public class CoverCache { } } - if (inputStream == null && (CoverCache.mCoverLoadMode & CoverCache.COVER_MODE_ANDROID) != 0 && song.id >= 0) { + if (inputStream == null && (CoverCache.mCoverLoadMode & CoverCache.COVER_MODE_ANDROID) != 0) { Uri uri = Uri.parse("content://media/external/audio/media/" + song.id + "/albumart"); ContentResolver res = mContext.getContentResolver(); inputStream = res.openInputStream(uri);