Viewing a particular SVG image causes Safari+SVG to stop responding.

• Component: SVG
• Status: RESOLVED FIXED

[Tobin Richard]

Attempting to open http://upload.wikimedia.org/wikipedia/en/0/01/Eye.svg (linked
to from http://en.wikipedia.org/wiki/Image:Eye.svg) causes Safari+SVG (latest
build from 2005-09-03) to stop responding.

[Eric Seidel (no email)]

Wow, thanks for the bug!

It looks like
SVGPathParser::parseSVG( const QString &s, bool process )

is just not robust enough.  This particular SVG has a non-ascii character right in the middle of a pata data 
attribute string, and it's causing parseSVG to just loop forever.  I'll talk this one out with the KSVG2 guys 
and see if we can't come up with a fix.

[Rob Buis]

(In reply to comment #0) 
> Attempting to open http://upload.wikimedia.org/wikipedia/en/0/01/Eye.svg 
(linked 
> to from http://en.wikipedia.org/wiki/Image:Eye.svg) causes Safari+SVG (latest 
> build from 2005-09-03) to stop responding. 
 
Good catch! I'll attach a patch that seems to work. I am checking whether 
just skipping the invalid char is according to svg... 
Cheers, 
 
Rob. 

[Rob Buis]

Created attachment 4169 [details]
Stop da loop

[Rob Buis]

Created attachment 4174 [details]
stop da loop II

[Eric Seidel (no email)]

Comment on attachment 4174 [details]
stop da loop II

Looks fine. r=me

[Rob Buis]

Created attachment 4175 [details]
More informative error reporting

[Eric Seidel (no email)]

Thanks for the great bug!