[FFmpeg-devel] [PATCH] mxfdec: Add missing free.

[Tomas Härdin]

On Sun, 2012-01-15 at 10:08 +0100, Reimar Döffinger wrote:
> Should fix valgrind FATE.
> 
> Signed-off-by: Reimar Döffinger <Reimar.Doeffinger at gmx.de>
> ---
>  libavformat/mxfdec.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c
> index 10512ef..ac871c3 100644
> --- a/libavformat/mxfdec.c
> +++ b/libavformat/mxfdec.c
> @@ -1915,6 +1915,7 @@ static int mxf_read_close(AVFormatContext *s)
>  
>      for (i = 0; i < mxf->nb_index_tables; i++) {
>          av_freep(&mxf->index_tables[i].segments);
> +        av_freep(&mxf->index_tables[i].ptses);
>          av_freep(&mxf->index_tables[i].fake_index);
>      }
>      av_freep(&mxf->index_tables);

Yikes! OK of course.
Does fate-demux run with valgrind now? Didn't it before, or is it
something that has to be done manually?

/Tomas