[FFmpeg-devel] [PATCH] rtsp: Don't store RTSPStream in AVStream->priv_data

Luca Barbato lu_zero
Thu Feb 3 01:14:54 CET 2011


On 02/03/2011 12:15 AM, Janne Grunau wrote:
> On Wed, Feb 02, 2011 at 11:30:07AM +0100, Luca Barbato wrote:
>> On 02/02/2011 10:08 AM, Martin Storsj? wrote:
>>> For mpegts in RTP, there isn't a direct mapping between RTSPStreams
>>> and AVStreams, and the RTSPStream isn't ever stored in
>>> AVStream->priv_data, which was earlier leaked. The fix for this
>>> leak, in ea7f080749d68a431226ce196014da38761a0d82, lead to
>>> double frees for other, normal RTP streams.
>>>
>>> This patch avoids storing RTSPStreams in AVStream->priv_data, thus
>>> avoiding the double free. The RTSPStreams are always available via
>>> RTSPState->rtsp_streams anyway.
>>
>> Looks fine, thanks for implementing and testing it =)
> 
> queued

Queued as well.

lu

-- 

Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu_zero




More information about the ffmpeg-devel mailing list