fix a nasty bug when deleting a stream from the playlist

src/directory.c

@@ -1229,7 +1229,7 @@ void initMp3Directory() {
 Song * getSongDetails(char * file, char ** shortnameRet, 
 		Directory ** directoryRet)
 {
-	void * song;
+	void * song = NULL;
 	Directory * directory;
 	char * dir = NULL;
 	char * dup = strdup(file);

src/playlist.c

@@ -138,6 +138,12 @@ void initPlaylist() {
 }
 
 void finishPlaylist() {
+        int i;
+        for(i=0;i<playlist.length;i++) {
+		if(playlist.songs[i]->type == SONG_TYPE_URL) {
+			/*freeJustSong(playlist.songs[i]);*/
+		}
+        }
 	free(playlist.songs);
 	playlist.songs = NULL;
 	free(playlist.order);
@@ -151,7 +157,7 @@ int clearPlaylist(FILE * fp) {
 
 	for(i=0;i<playlist.length;i++) {
 		if(playlist.songs[i]->type == SONG_TYPE_URL) {
-			free(playlist.songs[i]);
+			/*freeJustSong(playlist.songs[i]);*/
 		}
 		playlist.songs[i] = NULL;
 	}
@@ -584,6 +590,10 @@ int deleteFromPlaylist(FILE * fp, int song) {
 		}
 	}
 
+	if(playlist.songs[song]->type == SONG_TYPE_URL) {
+		freeJustSong(playlist.songs[song]);
+	}
+
 	/* delete song from songs array */
 	for(i=song;i<playlist.length-1;i++) {
 		playlist.songs[i] = playlist.songs[i+1];
@@ -600,9 +610,6 @@ int deleteFromPlaylist(FILE * fp, int song) {
 		if(playlist.order[i]>song) playlist.order[i]--;
 	}
 	/* now take care of other misc stuff */
-	if(playlist.songs[playlist.length-1]->type == SONG_TYPE_URL) {
-		freeJustSong(playlist.songs[playlist.length-1]);
-	}
 	playlist.songs[playlist.length-1] = NULL;
 	playlist.length--;
 

src/song.h

@@ -31,8 +31,8 @@
 #include "list.h"
 
 typedef enum {
-	SONG_TYPE_FILE,
-	SONG_TYPE_URL
+	SONG_TYPE_FILE = 1,
+	SONG_TYPE_URL = 2
 } SONG_TYPE;
 
 typedef struct _Song {