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

2015-08-25 Thread Stefano Sabatini
On date Friday 2015-08-21 12:21:05 +0200, Carl Eugen Hoyos encoded: > 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 > > Please review, Carl Eugen > diff --git a/libav

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

2015-08-23 Thread Ganesh Ajjanagadde
On Sun, Aug 23, 2015 at 12:04 PM, Ganesh Ajjanagadde wrote: > On Fri, Aug 21, 2015 at 6:21 AM, Carl Eugen Hoyos 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.

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

2015-08-23 Thread Ganesh Ajjanagadde
On Fri, Aug 21, 2015 at 6:21 AM, Carl Eugen Hoyos 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 sti

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

2015-08-21 Thread Carl Eugen Hoyos
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 Please review, Carl Eugen diff --git a/libavformat/segment.c b/libavformat/segment.c index 0d66211..9345dc9 100644 --- a/libavfor