On Wed, Nov 30, 2016 at 12:21:41AM +0100, Andreas Cadhalpun wrote:
> On 29.11.2016 02:49, Michael Niedermayer wrote:
> > On Tue, Nov 29, 2016 at 12:33:17AM +0100, Andreas Cadhalpun wrote:
> >> Signed-off-by: Andreas Cadhalpun <andreas.cadhal...@googlemail.com>
> >> ---
> >>  libavformat/avidec.c | 2 ++
> >>  1 file changed, 2 insertions(+)
> > 
> > If previous extradata is freed it should probably print a warning or
> > error out
> 
> That's a good idea, updated patch is attached.
> 
> Best regards,
> Andreas
> 

>  avidec.c |    8 ++++++++
>  1 file changed, 8 insertions(+)
> 88b8c27a72e7ec4519e92eee5a3a2764a657a823  
> 0001-avidec-fix-leaking-extradata.patch
> From e6a4fc92a99e2e3789730e8e122b39255d186c1f Mon Sep 17 00:00:00 2001
> From: Andreas Cadhalpun <andreas.cadhal...@googlemail.com>
> Date: Tue, 29 Nov 2016 00:28:55 +0100
> Subject: [PATCH] avidec: fix leaking extradata

LGTM

thx

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

When the tyrant has disposed of foreign enemies by conquest or treaty, and
there is nothing more to fear from them, then he is always stirring up
some war or other, in order that the people may require a leader. -- Plato

Attachment: signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to