From c9aec3e8666628c87b55657ae506a1bd4d27859d Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Thu, 24 Apr 2014 08:13:20 +0200 Subject: DatabaseCommands: fix crash on "list base" The string "base" is understood by locate_parse_type(), but not by listAllUniqueTags(). The special tag type LOCATE_TAG_BASE_TYPE causes a crash in PrintUniqueTag(). --- src/command/DatabaseCommands.cxx | 9 ++------- src/db/DatabasePrint.cxx | 2 ++ 2 files changed, 4 insertions(+), 7 deletions(-) (limited to 'src') diff --git a/src/command/DatabaseCommands.cxx b/src/command/DatabaseCommands.cxx index 919838e67..7126fc855 100644 --- a/src/command/DatabaseCommands.cxx +++ b/src/command/DatabaseCommands.cxx @@ -173,17 +173,12 @@ handle_list(Client &client, int argc, char *argv[]) { unsigned tagType = locate_parse_type(argv[1]); - if (tagType == TAG_NUM_OF_ITEM_TYPES) { + if (tagType >= TAG_NUM_OF_ITEM_TYPES && + tagType != LOCATE_TAG_FILE_TYPE) { command_error(client, ACK_ERROR_ARG, "\"%s\" is not known", argv[1]); return CommandResult::ERROR; } - if (tagType == LOCATE_TAG_ANY_TYPE) { - command_error(client, ACK_ERROR_ARG, - "\"any\" is not a valid return tag type"); - return CommandResult::ERROR; - } - SongFilter *filter; if (argc == 3) { /* for compatibility with < 0.12.0 */ diff --git a/src/db/DatabasePrint.cxx b/src/db/DatabasePrint.cxx index 2a13b28e6..781866272 100644 --- a/src/db/DatabasePrint.cxx +++ b/src/db/DatabasePrint.cxx @@ -261,6 +261,8 @@ listAllUniqueTags(Client &client, unsigned type, std::ref(client), _1); return db->Visit(selection, f, error); } else { + assert(type < TAG_NUM_OF_ITEM_TYPES); + using namespace std::placeholders; const auto f = std::bind(PrintUniqueTag, std::ref(client), (TagType)type, _1); -- cgit v1.2.3