[MPlayer-dev-eng] Re: [PATCH] URL escaping fix

[Alban Bedel]

On Tue, 22 Nov 2005 23:29:03 +0100
Alban Bedel <albeu at free.fr> wrote:

> 
> Hi all,
> 
> i regulary had some pb with some url and finally tracked it down.
> It show up on url who have for ex. a & encoded in them for ex:
> 
> http://wiki.null.sec/mp3.php?pls=%5BTroimah%20%26%20Panik130%5D%20Demo-Aufnahmen&recurse=1
> 
> mplayer escape the url, but to avoid escaping escape it first unescape
> the whole url. So here it transform the %26 in & and then doesn't escape
> the & again :(
> 
> The attached patch fix the pb by first "splitting" the url on the special
> chars and then doing the unescape/escape on each "part". That way escaped
> special chars get throught as they should.

Here is an updated version also fixing the problem with ip6 address
pointed out by qwen dux.

This will be committed soon unless someone object.

	Albeu
-------------- next part --------------
A non-text attachment was scrubbed...
Name: url_esc.diff
Type: application/octet-stream
Size: 4753 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20051125/85859554/attachment.obj>