diff options
| author | Max Kellermann <max@duempel.org> | 2012-08-07 23:37:38 +0200 |
|---|---|---|
| committer | Max Kellermann <max@duempel.org> | 2012-08-07 23:45:00 +0200 |
| commit | 8855efebc0488555b5a985589ee1f070c352c687 (patch) | |
| tree | a94dc07f5b1a446e4cdf311afb8aefd11e0e2b71 /src/command.c | |
| parent | 7725577a5347958de47476f4659c74094d18de32 (diff) | |
| download | mpd-8855efebc0488555b5a985589ee1f070c352c687.tar.gz mpd-8855efebc0488555b5a985589ee1f070c352c687.tar.xz mpd-8855efebc0488555b5a985589ee1f070c352c687.zip | |
locate: don't allow empty list
Diffstat (limited to 'src/command.c')
| -rw-r--r-- | src/command.c | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/src/command.c b/src/command.c index b1d222250..4631e7697 100644 --- a/src/command.c +++ b/src/command.c @@ -679,10 +679,7 @@ handle_playlistfind(struct client *client, int argc, char *argv[]) struct locate_item_list *list = locate_item_list_parse(argv + 1, argc - 1, false); - if (list == NULL || list->length == 0) { - if (list != NULL) - locate_item_list_free(list); - + if (list == NULL) { command_error(client, ACK_ERROR_ARG, "incorrect arguments"); return COMMAND_RETURN_ERROR; } @@ -700,10 +697,7 @@ handle_playlistsearch(struct client *client, int argc, char *argv[]) struct locate_item_list *list = locate_item_list_parse(argv + 1, argc - 1, true); - if (list == NULL || list->length == 0) { - if (list != NULL) - locate_item_list_free(list); - + if (list == NULL) { command_error(client, ACK_ERROR_ARG, "incorrect arguments"); return COMMAND_RETURN_ERROR; } |