From aeedcd66d699fa8e64df5ca75cf0d2f2c1957e08 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Sat, 3 Oct 2009 18:21:15 +0200 Subject: [PATCH] screen_artist: check cursor position in cmd() method Fix segmentation faults when the list is empty. --- src/screen_artist.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/src/screen_artist.c b/src/screen_artist.c index 5c0ff30..95e4d22 100644 --- a/src/screen_artist.c +++ b/src/screen_artist.c @@ -470,6 +470,9 @@ screen_artist_cmd(struct mpdclient *c, command_t cmd) case CMD_PLAY: switch (mode) { case LIST_ARTISTS: + if (browser.lw->selected >= artist_list->len) + return true; + selected = g_ptr_array_index(artist_list, browser.lw->selected); open_album_list(c, g_strdup(selected)); @@ -603,6 +606,9 @@ screen_artist_cmd(struct mpdclient *c, command_t cmd) case CMD_ADD: switch(mode) { case LIST_ARTISTS: + if (browser.lw->selected >= artist_list->len) + return true; + for (unsigned i = browser.lw->selected_start; i <= browser.lw->selected_end; ++i) { selected = g_ptr_array_index(artist_list, i); -- 2.30.2