Re: [Gnash-dev] Improve URL parsing using a sane comparison. (patch)

[annonygmouse]

Bastiaan Jacques wrote:
> On Sunday 16 July 2006 09:53, annonygmouse wrote:
>
> > I've found lots of URLs that the plugin didn't know how to parse
> > "Can't parse URL".
>
>
> Could you provide an example of a URL that would fail to be parsed?

Certainly, here's what I've done:

Download CVS version.
Add some logging (see plugin.cpp attached, lines 540 <--> 546).
run
firefox
"http://www.periodistadigital.com/img/promos/repsol120x80mapa.swf?urlflash=http://www.repsolypf.com/comunes/psolicitada.asp?URL=http://www.repsolypf.com/esp/ventajasdelportal/promociones/promoruta/introduccion.asp?PaginaID=102884&email=120x80Home&cuenta=Periodistadigital&Envio=1282";
21:14:29: xEmbed supported in this Mozilla version
21:14:29: Gtk2+ supported in this Mozilla version
21:14:29: resizeWindow: Width = 1024, Height = 605
21:14:29: NewStream: The full URL is
http://www.periodistadigital.com/img/promos/repsol120x80mapa.swf?urlflash=http://www.repsolypf.com/comunes/psolicitada.asp?URL=http://www.repsolypf.com/esp/ventajasdelportal/promociones/promoruta/introduccion.asp?PaginaID=102884&email=120x80Home&cuenta=Periodistadigital&Envio=1282
21:14:29: NewStream: Before end=opts.find("&",start)
Opts=urlflash=http://www.repsolypf.com/comunes/psolicitada.asp?URL=http://www.repsolypf.com/esp/ventajasdelportal/promociones/promoruta/introduccion.asp?PaginaID=102884&email=120x80Home&cuenta=Periodistadigital&Envio=1282 




end=281
21:14:29: NewStream: After end=opts.find("&",start)
Opts=urlflash=http://www.repsolypf.com/comunes/psolicitada.asp?URL=http://www.repsolypf.com/esp/ventajasdelportal/promociones/promoruta/introduccion.asp?PaginaID=102884&email=120x80Home&cuenta=Periodistadigital&Envio=1282 




end=163
21:14:29: NewStream: If end was <=0 now shold be opts.size()... let's
see: end=opts.find("&",start)
Opts=urlflash=http://www.repsolypf.com/comunes/psolicitada.asp?URL=http://www.repsolypf.com/esp/ventajasdelportal/promociones/promoruta/introduccion.asp?PaginaID=102884&email=120x80Home&cuenta=Periodistadigital&Envio=1282 




end=163
21:14:29: NewStream: Before end=opts.find("&",start)
Opts=email=120x80Home&cuenta=Periodistadigital&Envio=1282 end=164
21:14:29: NewStream: After end=opts.find("&",start)
Opts=email=120x80Home&cuenta=Periodistadigital&Envio=1282 end=16
21:14:29: NewStream: If end was <=0 now shold be opts.size()... let's
see: end=opts.find("&",start)
Opts=email=120x80Home&cuenta=Periodistadigital&Envio=1282 end=16
21:14:29: NewStream: Before end=opts.find("&",start)
Opts=cuenta=Periodistadigital&Envio=1282 end=17
21:14:29: NewStream: After end=opts.find("&",start)
Opts=cuenta=Periodistadigital&Envio=1282 end=24
21:14:29: NewStream: If end was <=0 now shold be opts.size()... let's
see: end=opts.find("&",start) Opts=cuenta=Periodistadigital&Envio=1282
end=24
21:14:29: NewStream: Before end=opts.find("&",start) Opts=Envio=1282 end=25
21:14:29: NewStream: After end=opts.find("&",start) Opts=Envio=1282 end=-1
21:14:29: NewStream: If end was <=0 now shold be opts.size()... let's
see: end=opts.find("&",start) Opts=Envio=1282 end=-1
21:14:29: ERROR: Can't parse URL!


Hope it helps.
Sebastià

plugin.cpp.gz
Description: GNU Zip compressed data

config.log.gz
Description: GNU Zip compressed data