[FFmpeg-devel] [PATCH] Fix redirector implementation

[Luca Abeni]

Hi Michael,

Michael Niedermayer wrote:
[...]
>> I am thinking about a more radical approach: basically, doing a
>> *s = *ic;
>> at the end of redir_read_header()... Do you see any obvious problems with it?
> 
> no but that doesnt say much, there still could be obvious problems :)
> 
>> I am going to test this idea, and will send a patch this afternoon.
> 
> ok

And here is the patch... It's even simpler than the first one :)
Tested with ffserver's redirection from http to rtsp, and it works.

I tried to detect memory leaks with mtrace(), and there are no changes
respect to ffmpeg from svn (mtrace shows some "non freed" memory on
unpatched ffmpeg. I did not investigate this, but maybe it is expected).


			Thanks,
				Luca
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: fix_redir-5.diff
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071128/cfef4efd/attachment.txt>