[FFmpeg-devel] [PATCH] ffplay: fix mem leak when opening input or parsing options fail.

Michael Niedermayer michaelni at gmx.at
Sat Nov 22 15:43:22 CET 2014


On Sat, Nov 22, 2014 at 02:58:01PM +0100, Marton Balint wrote:
> 
> 
> On Fri, 21 Nov 2014, Benoit Fouet wrote:
> 
> >---
> >ffplay.c | 5 +++--
> >1 file changed, 3 insertions(+), 2 deletions(-)
> >
> >diff --git a/ffplay.c b/ffplay.c
> >index f79161d..1914a66 100644
> >--- a/ffplay.c
> >+++ b/ffplay.c
> >@@ -3169,8 +3169,9 @@ static int read_thread(void *arg)
> >        stream_component_close(is, is->video_stream);
> >    if (is->subtitle_stream >= 0)
> >        stream_component_close(is, is->subtitle_stream);
> >-    if (is->ic) {
> >-        avformat_close_input(&is->ic);
> >+    if (ic) {
> >+        avformat_close_input(&ic);
> >+        is->ic = NULL;
> >    }
> >
> >    if (ret != 0) {
> >-- 
> >2.2.0.rc2.23.gca0107e
> 
> LGTM, altough if avformat_open_input fails, it frees the context
> according to docs, so I guess this is only really needed for the
> option parsing failure.

applied

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Avoid a single point of failure, be that a person or equipment.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20141122/d38a5891/attachment.asc>


More information about the ffmpeg-devel mailing list