[FFmpeg-devel] [PATCH] j2kdec: fix context tile memleak.

[Reimar Döffinger]

On Fri, Jan 13, 2012 at 06:02:37AM +0100, Michael Niedermayer wrote:
> On Fri, Jan 13, 2012 at 02:06:55AM +0100, Clément Bœsch wrote:
> > ---
> >  libavcodec/j2kdec.c |    1 +
> >  1 files changed, 1 insertions(+), 0 deletions(-)
> > 
> > diff --git a/libavcodec/j2kdec.c b/libavcodec/j2kdec.c
> > index 1f4cdeb..8d8d9e1 100644
> > --- a/libavcodec/j2kdec.c
> > +++ b/libavcodec/j2kdec.c
> > @@ -1063,6 +1063,7 @@ static av_cold int decode_end(AVCodecContext *avctx)
> >  
> >      if (s->picture.data[0])
> >          avctx->release_buffer(avctx, &s->picture);
> > +    av_freep(&s->tile);
> 
> this should have been freed by cleanup(), apparently its not ...

Nothing is freed on certain errors, I expect my patch will fix it.