diff options
author | Max Kellermann <max@duempel.org> | 2010-01-18 11:05:15 +0100 |
---|---|---|
committer | Max Kellermann <max@duempel.org> | 2010-06-30 23:14:43 +0200 |
commit | 814daac5bae1e7e3a67cd7ed727bd1fe6b9a3889 (patch) | |
tree | 1bd45c9a50b2436edf5287a27550bd7bf6e91f33 | |
parent | 0d03bdce6d590429143e821e2bcf34142b57b8fa (diff) | |
download | mpd-814daac5bae1e7e3a67cd7ed727bd1fe6b9a3889.tar.gz mpd-814daac5bae1e7e3a67cd7ed727bd1fe6b9a3889.tar.xz mpd-814daac5bae1e7e3a67cd7ed727bd1fe6b9a3889.zip |
decoder/ffmpeg: free AVFormatContext on error
Fix a memory leak in some code paths.
-rw-r--r-- | NEWS | 1 | ||||
-rw-r--r-- | src/decoder/ffmpeg_plugin.c | 4 |
2 files changed, 5 insertions, 0 deletions
@@ -4,6 +4,7 @@ ver 0.15.11 (2010/??/??) * decoders: - mp4ff: support tags "album artist", "albumartist", "band" - mikmod: fix memory leak + - ffmpeg: free AVFormatContext on error * playlist: emit IDLE_OPTIONS when resetting single mode * listen: make get_remote_uid() work on BSD diff --git a/src/decoder/ffmpeg_plugin.c b/src/decoder/ffmpeg_plugin.c index 2a46601f4..2e06c13f8 100644 --- a/src/decoder/ffmpeg_plugin.c +++ b/src/decoder/ffmpeg_plugin.c @@ -192,12 +192,14 @@ ffmpeg_helper(const char *uri, struct input_stream *input, if (av_find_stream_info(format_context)<0) { g_warning("Couldn't find stream info\n"); + av_close_input_file(format_context); return false; } audio_stream = ffmpeg_find_audio_stream(format_context); if (audio_stream == -1) { g_warning("No audio stream inside\n"); + av_close_input_file(format_context); return false; } @@ -209,11 +211,13 @@ ffmpeg_helper(const char *uri, struct input_stream *input, if (!codec) { g_warning("Unsupported audio codec\n"); + av_close_input_file(format_context); return false; } if (avcodec_open(codec_context, codec)<0) { g_warning("Could not open codec\n"); + av_close_input_file(format_context); return false; } |