On Fri, Aug 21, 2015 at 6:21 AM, Carl Eugen Hoyos <ceho...@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@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel