mustang2247 / svgweb

Automatically exported from code.google.com/p/svgweb
Other
0 stars 0 forks source link

Semi-legal regex is used in the code. #466

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Running svg.js through Rhino (don't ask) revealed that it contains a
questionable regex rejected by Rhino: /\/([^/]*)$/ --- it contains an
unescaped slash. Just to be on a safe side it should be escaped like this:
/\/([^\/]*)$/

This regex is encountered inside three times. It is a trivial change. I
don't know if it has any clashes with browser-based JS engines, but it
doesn't hurt to be in the green on this issue.

Original issue reported on code.google.com by eugene.l...@gmail.com on 4 Apr 2010 at 1:52

GoogleCodeExporter commented 8 years ago
Thanks for finding this Eugene!

Original comment by bradneub...@gmail.com on 5 Apr 2010 at 5:24

GoogleCodeExporter commented 8 years ago
This issue was closed by revision r1097.

Original comment by bradneub...@gmail.com on 8 Apr 2010 at 9:54