[FFmpeg-soc] [PATCH] RTSP/tunneling: clean up properly on failures

Martin Storsjö martin at martin.st
Fri Jun 18 19:57:05 CEST 2010


On Fri, 18 Jun 2010, Josh Allmann wrote:

> On 18 June 2010 00:45, Martin Storsjö <martin at martin.st> wrote:
> > Hi,
> >
> > I noticed that the GET URLContext can be leaked if some of the later
> > operations fail before assigning the handles to rt->rtsp_hd{,_out}. This
> > patch should fix the issue.
> >
> > Josh, do you think this is ok, or would you rather assign the handles
> > directly to rt->rtsp_hd without using the local variables?
> >
> 
> Doesn't the jump to fail: clean things up? If I'm missing something
> here, then it looks OK.

Applied this fix, feel free to propose other ways to reorganize it to make 
it cleaner (e.g. by assigning the handles directly to rt->rtsp_hd, so the 
normal cleanup path can take care of it.

// Martin


More information about the FFmpeg-soc mailing list