From e1001491240f5925d5f75673552e297c3244e107 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Sat, 24 Jan 2009 15:27:09 +0100 Subject: locate: initialize the whole visited_types array The declaration initialized only the first element. Initialize the whole array with memset() instead. --- src/locate.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) (limited to 'src/locate.c') diff --git a/src/locate.c b/src/locate.c index 85fe297a5..2833a5f8b 100644 --- a/src/locate.c +++ b/src/locate.c @@ -130,7 +130,7 @@ locate_tag_search(const struct song *song, enum tag_type type, const char *str) int i; char *duplicate; bool ret = false; - bool visited_types[TAG_NUM_OF_ITEM_TYPES] = { false }; + bool visited_types[TAG_NUM_OF_ITEM_TYPES]; if (type == LOCATE_TAG_FILE_TYPE || type == LOCATE_TAG_ANY_TYPE) { char *uri, *p; @@ -149,6 +149,8 @@ locate_tag_search(const struct song *song, enum tag_type type, const char *str) if (!song->tag) return false; + memset(visited_types, 0, sizeof(visited_types)); + for (i = 0; i < song->tag->numOfItems && !ret; i++) { visited_types[song->tag->items[i]->type] = true; if (type != LOCATE_TAG_ANY_TYPE && @@ -189,7 +191,7 @@ static bool locate_tag_match(const struct song *song, enum tag_type type, const char *str) { int i; - bool visited_types[TAG_NUM_OF_ITEM_TYPES] = { false }; + bool visited_types[TAG_NUM_OF_ITEM_TYPES]; if (type == LOCATE_TAG_FILE_TYPE || type == LOCATE_TAG_ANY_TYPE) { char *uri = song_get_uri(song); @@ -206,6 +208,8 @@ locate_tag_match(const struct song *song, enum tag_type type, const char *str) if (!song->tag) return false; + memset(visited_types, 0, sizeof(visited_types)); + for (i = 0; i < song->tag->numOfItems; i++) { visited_types[song->tag->items[i]->type] = true; if (type != LOCATE_TAG_ANY_TYPE && -- cgit v1.2.3