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

[Janne Grunau]

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

Janne