[MPlayer-dev-eng] url.c (maybe) 10l bug

Bertrand Baudet bertrand_baudet at yahoo.com
Wed Apr 7 22:35:54 CEST 2004


Fixed

On Wednesday 07 April 2004 00:29, Bertrand Baudet wrote:
> On Tuesday 06 April 2004 14:17, Alex Beregszaszi wrote:
> > Hi,
> >
> > > the upper block accepts '%' as-is unescaped, but the lower block then
> > > uses '%' to escape special chars. i guess that '%' should be escaped
> > > too...
> > >
> > > or maybe the '%' is allowed here to let this function work on already
> > > escaped URIs? it makes sence, but still has a buggy side-effect on
> > > unescaped strings containing '%' char...
> > >
> > > if no objection, Attila or Diego will commit a fix :))
> >
> > Any fixes on this? Bertrand, please comment!
>
> Looks like Arpi is right.
>
> From the RFC 1738:
>
> "The character "%" is unsafe because it is used for encodings of other
> characters."
> And
> "All unsafe characters must always be encoded within a URL."
>
> I'm going to commit a fix tomorrow.
>
> Bertrand
>
> _______________________________________________
> MPlayer-dev-eng mailing list
> MPlayer-dev-eng at mplayerhq.hu
> http://mplayerhq.hu/mailman/listinfo/mplayer-dev-eng




More information about the MPlayer-dev-eng mailing list