[FFmpeg-devel] [PATCH]lavf/segment: Fix memleak

Ganesh Ajjanagadde gajjanag at mit.edu
Sun Aug 23 18:04:58 CEST 2015


On Fri, Aug 21, 2015 at 6:21 AM, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
> Hi!
>
> Attached patch fixes a memleak for me, reproducible with:
> $ valgrind --leak-check=full ./ffmpeg_g -i
> fate-suite/lena.pnm -reset_timestamps 1 -f segment out%1d.avi

It fixes a memleak for me, but note that valgrind output is still not clean:
3 errors from 2 contexts becomes
2 errors from 1 context.

Are you sure nothing else needs to be freed?
I am unfortunately not familiar with this code at all,
so I don't know what needs to be done.

>
> Please review, Carl Eugen
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>


More information about the ffmpeg-devel mailing list