From 6726f07268feea32b99988b7582172b9e81d0f48 Mon Sep 17 00:00:00 2001 From: Adrian Ulrich Date: Sun, 24 Aug 2014 15:24:56 +0200 Subject: [PATCH] fix crash if creating a playlist fails --- src/ch/blinkenlights/android/vanilla/Playlist.java | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/ch/blinkenlights/android/vanilla/Playlist.java b/src/ch/blinkenlights/android/vanilla/Playlist.java index 0bf886d9..d27720cb 100644 --- a/src/ch/blinkenlights/android/vanilla/Playlist.java +++ b/src/ch/blinkenlights/android/vanilla/Playlist.java @@ -107,7 +107,12 @@ public class Playlist { ContentValues values = new ContentValues(1); values.put(MediaStore.Audio.Playlists.NAME, name); Uri uri = resolver.insert(MediaStore.Audio.Playlists.EXTERNAL_CONTENT_URI, values); - id = Long.parseLong(uri.getLastPathSegment()); + /* Creating the playlist may fail due to race conditions or silly + * android bugs (i am looking at you, kitkat!). In this case, id will stay -1 + */ + if (uri != null) { + id = Long.parseLong(uri.getLastPathSegment()); + } } else { // We are overwriting an existing playlist. Clear existing songs. Uri uri = MediaStore.Audio.Playlists.Members.getContentUri("external", id);