[Ffmpeg-devel] [PATCH] Fix leak of AVPacket data in ffserver

Bryan Mayland bmayland
Mon Jan 23 23:38:13 CET 2006


Bryan Mayland wrote:
> I was noticing that ffserver memory usage steadily grows over time.  
> There are a couple reasons for this, but here is the big one.  When in 
> the HTTPSTATE_SEND_DATA state, we're basically in a 
> av_read_frame()/av_write_frame() loop.  If the av_write_frame doesn't 
> need a flush (which is the case when writing low bitrate [96kbit] asf 
> streams), nothing is written to the dynbuff and the code loops back to 
> read more frames to fill the outgoing buffer.  In this case however, 
> the data attached to the AVPacket is not freed with a call to 
> av_free_packet.
> This patch fixes that in the common case as well as in the case a 
> dynbuff could not be allocated.  NOTE:  This patch must be applied 
> *after* my ffserver_warning_clean2.diff patch.
    Dammit.  Helps if I attach the right patch.  This one attached 
correctly fixes both conditions.
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: ffserver_packet_leak2.diff
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20060123/cb82b008/attachment.asc>



More information about the ffmpeg-devel mailing list