[FFmpeg-devel] [PATCH] mxfdec: Add missing free.
Tomas Härdin
tomas.hardin at codemill.se
Mon Jan 16 13:30:10 CET 2012
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
More information about the ffmpeg-devel
mailing list