[FFmpeg-devel] [PATCH 2/2] ffserver: fix streams memleak when closing a connection.
Michael Niedermayer
michaelni at gmx.at
Thu Nov 15 11:48:22 CET 2012
On Thu, Nov 15, 2012 at 09:42:21AM +0100, Clément Bœsch wrote:
> On Wed, Nov 14, 2012 at 05:18:45PM +0100, Michael Niedermayer wrote:
> > On Tue, Nov 13, 2012 at 04:35:18PM +0100, Clément Bœsch wrote:
> > > From: Clément Bœsch <clement.boesch at smartjog.com>
> > >
> > > ---
> > > ffserver.c | 1 +
> > > 1 file changed, 1 insertion(+)
> > >
> > > diff --git a/ffserver.c b/ffserver.c
> > > index 0d2a8a7..28a614f 100644
> > > --- a/ffserver.c
> > > +++ b/ffserver.c
> > > @@ -929,6 +929,7 @@ static void close_connection(HTTPContext *c)
> > >
> > > for(i=0; i<ctx->nb_streams; i++)
> > > av_free(ctx->streams[i]);
> > > + av_free(ctx->streams);
> >
> > should be av_freep()
> >
>
> Well, a few lines below the whole context is completely destroyed so it
> doesn't really matter, but changed anyway since it doesn't hurt either.
>
> Also note that this isn't enough to fix the leak: adding a
> av_free(ctx->priv_data) fixes the real problem. Both streams and priv_data
> are locally allocated, the rest of the structure is shared between the
> connections so we can't use avformat_free_context() or similar.
>
> New patch attached.
>
> --
> Clément B.
> ffserver.c | 2 ++
> 1 file changed, 2 insertions(+)
> 1724952ddd4c928d7bb09278c5453a019a063e21 0001-ffserver-fix-streams-and-priv_data-memleaks-when-clo.patch
> From 93df34bcc708ab17fcd2da231e63ac51bdb2c1b2 Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?Cl=C3=A9ment=20B=C5=93sch?= <clement.boesch at smartjog.com>
> Date: Tue, 13 Nov 2012 17:32:35 +0200
> Subject: [PATCH] ffserver: fix streams and priv_data memleaks when closing a
> connection.
LGTM, thanks
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
No human being will ever know the Truth, for even if they happen to say it
by chance, they would not even known they had done so. -- Xenophanes
-------------- 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/20121115/695fedb7/attachment.asc>
More information about the ffmpeg-devel
mailing list