[FFmpeg-devel] [PATCH] ffserver rtsp bug fixes

Howard Chu hyc
Wed May 19 08:47:36 CEST 2010


Howard Chu wrote:
> Howard Chu wrote:
>> Howard Chu wrote:
>>> This patch plugs a number of per-session memory leaks in the rtsp code. Also
>>> replaced one instance of strchr() with memchr() to silence a valgrind warning
>>> about running off into the weeds (reading uninit'd memory). Unfortunately
>>> there are still a large number of those...
>>
>> All of my testing so far was with streams coming from an ffm feed. I just
>> found this bug while running valgrind using a stream read from a file. Streams
>> read from a file will reference freed memory because add_av_stream1() didn't
>> copy the extradata from the codec.
>>
>> Note that I still haven't gotten ffserver to successfully stream anything from
>> a file, there are plenty more problems left to chase down.
>
> It turns out that streaming just audio from FLV files was working fine. But
> the H264 video was encoded as the AVC1 subtype, and none of the sdp or rtp
> code handled this. This patch gets RTP working for me from an FLV (AVC1) file
> source.

Cleaned up, more compact version of the patch.
-- 
   -- Howard Chu
   CTO, Symas Corp.           http://www.symas.com
   Director, Highland Sun     http://highlandsun.com/hyc/
   Chief Architect, OpenLDAP  http://www.openldap.org/project/
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: avcfix.txt
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100518/b5cd4c64/attachment.txt>



More information about the ffmpeg-devel mailing list