diff options
author | Max Kellermann <max@duempel.org> | 2010-06-30 19:37:36 +0000 |
---|---|---|
committer | Max Kellermann <max@duempel.org> | 2010-06-30 19:37:36 +0000 |
commit | 77e6810c14e3b47f27a385e2fd9e8760710d0a99 (patch) | |
tree | 0a79d082dc5df1466b0132ed96fb809dddcf663c | |
parent | 5ebe33653c162a3a07a9a1fd71201e22fa152c6c (diff) | |
download | mpd-77e6810c14e3b47f27a385e2fd9e8760710d0a99.tar.gz mpd-77e6810c14e3b47f27a385e2fd9e8760710d0a99.tar.xz mpd-77e6810c14e3b47f27a385e2fd9e8760710d0a99.zip |
decoder/mikmod: fix memory leak
The return value of Player_LoadTitle() is allocated with malloc(), and
must be freed by the caller.
Diffstat (limited to '')
-rw-r--r-- | NEWS | 1 | ||||
-rw-r--r-- | src/decoder/mikmod_plugin.c | 6 |
2 files changed, 5 insertions, 2 deletions
@@ -3,6 +3,7 @@ ver 0.15.11 (2010/??/??) - ape: support album artist * decoders: - mp4ff: support tags "albumartist", "band" + - mikmod: fix memory leak ver 0.15.10 (2010/05/30) diff --git a/src/decoder/mikmod_plugin.c b/src/decoder/mikmod_plugin.c index 065c34319..f60dcbc61 100644 --- a/src/decoder/mikmod_plugin.c +++ b/src/decoder/mikmod_plugin.c @@ -219,10 +219,12 @@ static struct tag *modTagDup(const char *file) ret->time = 0; path2 = g_strdup(file); - title = g_strdup(Player_LoadTitle(path2)); + title = Player_LoadTitle(path2); g_free(path2); - if (title) + if (title) { tag_add_item(ret, TAG_ITEM_TITLE, title); + free(title); + } return ret; } |