From ba91edfee95a6198e2db9804fee9ddfeffa8a199 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Sun, 21 Sep 2008 22:45:04 +0200 Subject: browser: check if the selection is valid Don't call filelist_get() with an invalid index. --- src/screen_file.c | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'src/screen_file.c') diff --git a/src/screen_file.c b/src/screen_file.c index 9dd59001c..26b273972 100644 --- a/src/screen_file.c +++ b/src/screen_file.c @@ -62,6 +62,9 @@ handle_save(screen_t *screen, mpdclient_t *c) filelist_entry_t *entry; char *defaultname = NULL; + if (browser.lw->selected >= filelist_length(browser.filelist)) + return -1; + entry = filelist_get(browser.filelist, browser.lw->selected); if( entry && entry->entity ) { mpd_InfoEntity *entity = entry->entity; @@ -83,6 +86,9 @@ handle_delete(screen_t *screen, mpdclient_t *c) char *str, *buf; int key; + if (browser.lw->selected >= filelist_length(browser.filelist)) + return -1; + entry = filelist_get(browser.filelist, browser.lw->selected); if( entry==NULL || entry->entity==NULL ) return -1; -- cgit v1.2.3