playlist: convert assertion to check in TAG event handler

It is possible that playlist.current is reset before the TAG event handler playlist_tag_event() is called. Convert the assertion into a run-time check.

playlist: convert assertion to check in TAG event handler
8484c370 · Max Kellermann · 1d02318d · 8484c370
Commit 8484c370 authored Jan 21, 2009 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 2 deletions

playlist.c src/playlist.c +3 -2

No files found.
--- a/src/playlist.c
+++ b/src/playlist.c
@@ -123,10 +123,11 @@ playlist_tag_event(void)
 {
 	unsigned song;

-	if (playlist_state != PLAYLIST_STATE_PLAY)
+	if (playlist_state != PLAYLIST_STATE_PLAY ||
+	    playlist.current < 0)
 		return;

-	assert(playlist.current >= 0);
+	assert((unsigned)playlist.current < playlist.length);

 	song = playlist.order[playlist.current];
 	playlist.songMod[song] = playlist.version;