[FFmpeg-devel] [PATCH] avidec: fix leaking extradata

[Michael Niedermayer]

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.Cadhalpun at 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.Cadhalpun at 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20161130/a3c464fd/attachment.sig>