playlist_state: don't save "current" song when none is set

This patch fixes an assertion failure: Assertion `order < queue->length' failed. This happens when the state file is saved, when there is no "current" song: current==-1, and queue_order_to_position(-1) is called.

playlist_state: don't save "current" song when none is set
706614b0 · Max Kellermann · a641f562 · 706614b0
Commit 706614b0 authored Jun 29, 2009 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 2 deletions

playlist_state.c src/playlist_state.c +4 -2

No files found.
--- a/src/playlist_state.c
+++ b/src/playlist_state.c
@@ -68,9 +68,11 @@ playlist_state_save(FILE *fp, const struct playlist *playlist)
 			getPlayerElapsedTime());
 	} else {
 		fprintf(fp, "%s\n", PLAYLIST_STATE_FILE_STATE_STOP);
-		fprintf(fp, "%s%i\n", PLAYLIST_STATE_FILE_CURRENT,
+
+		if (playlist->current >= 0)
+			fprintf(fp, "%s%i\n", PLAYLIST_STATE_FILE_CURRENT,
 				queue_order_to_position(&playlist->queue,
-					playlist->current));
+							playlist->current));
 	}

 	fprintf(fp, "%s%i\n", PLAYLIST_STATE_FILE_RANDOM,