[FFmpeg-devel] [PATCH] boxblur: fix huge memleak in end_frame().
Michael Niedermayer
michaelni at gmx.at
Fri Nov 25 00:17:15 CET 2011
On Thu, Nov 24, 2011 at 08:45:26PM +0100, Clément Bœsch wrote:
> ---
> Hi,
>
> I have absolutely no idea why this is actually needed here (where are those
> buffer "referenced"? isn't it supposed to be free'd by the code allocating
> it?), but it seems to fix a huge memleak in this filter. Easily reproducible
> with something like: ./ffplay -f lavfi mandelbrot,boxblur=2:2; ~50MB/sec lost
> here.
As the filter uses the default start frame, the cleanup done by the
default end frame is needed.
does the following work for you ?
diff --git a/libavfilter/vf_boxblur.c b/libavfilter/vf_boxblur.c
index db1b579..ddb4b25 100644
--- a/libavfilter/vf_boxblur.c
+++ b/libavfilter/vf_boxblur.c
@@ -325,7 +325,7 @@ static void end_frame(AVFilterLink *inlink)
boxblur->temp);
avfilter_draw_slice(outlink, 0, inlink->h, 1);
- avfilter_end_frame(outlink);
+ avfilter_default_end_frame(inlink);
}
AVFilter avfilter_vf_boxblur = {
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
The bravest are surely those who have the clearest vision
of what is before them, glory and danger alike, and yet
notwithstanding go out to meet it. -- Thucydides
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111125/8ba72f68/attachment.asc>
More information about the ffmpeg-devel
mailing list