diff options
| author | Max Kellermann <max@duempel.org> | 2010-05-31 10:24:08 +0200 |
|---|---|---|
| committer | Max Kellermann <max@duempel.org> | 2010-05-31 10:24:08 +0200 |
| commit | de8f2739c2977ffb042bfd9c36630fe6f187901f (patch) | |
| tree | bbc6eee1359f9edb770fddbd52575d7c0412167e /src/decoder | |
| parent | 2c1c5888683197dba3477487ca49727fd48fbac6 (diff) | |
| download | mpd-de8f2739c2977ffb042bfd9c36630fe6f187901f.tar.gz mpd-de8f2739c2977ffb042bfd9c36630fe6f187901f.tar.xz mpd-de8f2739c2977ffb042bfd9c36630fe6f187901f.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 'src/decoder')
| -rw-r--r-- | src/decoder/mikmod_decoder_plugin.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/decoder/mikmod_decoder_plugin.c b/src/decoder/mikmod_decoder_plugin.c index 5cbfa23dd..50b46f2af 100644 --- a/src/decoder/mikmod_decoder_plugin.c +++ b/src/decoder/mikmod_decoder_plugin.c @@ -195,11 +195,13 @@ mikmod_decoder_tag_dup(const char *path_fs) tag->time = 0; - char *title = g_strdup(Player_LoadTitle(path2)); + char *title = Player_LoadTitle(path2); g_free(path2); - if (title) + if (title != NULL) { tag_add_item(tag, TAG_TITLE, title); + free(title); + } return tag; } |