From 0656660f22eb334a53d6435f187eba2592622a27 Mon Sep 17 00:00:00 2001 From: rubidium Date: Sun, 16 Oct 2011 19:56:47 +0000 Subject: [PATCH] (svn r23034) -Fix: make sure the custom playlists are 0 terminated --- src/music_gui.cpp | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/src/music_gui.cpp b/src/music_gui.cpp index 94e3f191bc..17f09f47c3 100644 --- a/src/music_gui.cpp +++ b/src/music_gui.cpp @@ -79,20 +79,24 @@ static byte * const _playlists[] = { /** * Validate a playlist. - * @param playlist the playlist to validate + * @param playlist The playlist to validate. + * @param last The last location in the list. */ -void ValidatePlaylist(byte *playlist) +void ValidatePlaylist(byte *playlist, const byte *last) { - while (*playlist != 0) { + while (*playlist != 0 && playlist <= last) { /* Song indices are saved off-by-one so 0 is "nothing". */ if (*playlist <= NUM_SONGS_AVAILABLE && !StrEmpty(GetSongName(*playlist - 1))) { playlist++; continue; } - for (byte *p = playlist; *p != 0; p++) { + for (byte *p = playlist; *p != 0 && p <= last; p++) { p[0] = p[1]; } } + + /* Make sure the list is null terminated. */ + *last = 0; } /** Initialize the playlists */ @@ -118,8 +122,8 @@ void InitializeMusic() _playlists[k + 1][j] = 0; } - ValidatePlaylist(_settings_client.music.custom_1); - ValidatePlaylist(_settings_client.music.custom_2); + ValidatePlaylist(_settings_client.music.custom_1, lastof(_settings_client.music.custom_1)); + ValidatePlaylist(_settings_client.music.custom_2, lastof(_settings_client.music.custom_2)); if (BaseMusic::GetUsedSet()->num_available < _music_wnd_cursong) { /* If there are less songs than the currently played song,