RESOLVED FIXED 13140
Crash after loading SVG fragment with XMLHttpRequest (Yosemite Map Geology Layer) 
https://bugs.webkit.org/show_bug.cgi?id=13140
Summary Crash after loading SVG fragment with XMLHttpRequest (Yosemite Map Geology La...
Andreas Neumann
Reported 2007-03-21 06:19:40 PDT
If you load this SVG file and click on the text (geology layer) an XMLHttpRequest() is generated and a SVG fragment added to the DOM. During or after loading Webkit crashes. I don't know if this bug is SVG related, DOM related or an XML parser problem. It somehow has to do with the content that is loaded, because it crashes on the geology layer, but not on other layers, like hydrology. A crashlog will be attached.
Attachments
crashlog for yosemite geology loading (24.91 KB, text/plain)
2007-03-21 06:24 PDT, Andreas Neumann 		no flags
Details
proposed fix (2.43 KB, patch)
2007-03-21 11:59 PDT, Alexey Proskuryakov 		no flags
DetailsFormatted DiffDiff
proposed fix (3.68 KB, patch)
2007-03-21 12:00 PDT, Alexey Proskuryakov 		no flags
DetailsFormatted DiffDiff
proposed fix (4.22 KB, patch)
2007-03-21 12:07 PDT, Alexey Proskuryakov 		mitz: review+
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Andreas Neumann
Comment 1 2007-03-21 06:24:28 PDT
Created attachment 13734 [details] crashlog for yosemite geology loading
David Kilzer (:ddkilzer)
Comment 2 2007-03-21 08:55:33 PDT
P1 crashers need a Radar, too! :)
Alexey Proskuryakov
Comment 3 2007-03-21 11:59:15 PDT
Created attachment 13743 [details] proposed fix I still don't really understand why xhr.responseXML has to create a full SVG (or XHTML) DOM, but ok...
Alexey Proskuryakov
Comment 4 2007-03-21 12:00:23 PDT
Created attachment 13744 [details] proposed fix Forgot to include chagnelogs.
Alexey Proskuryakov
Comment 5 2007-03-21 12:07:25 PDT
Created attachment 13745 [details] proposed fix So, I didn't even notice reportWarning next to reportError :( Thanks, Mitz!
mitz
Comment 6 2007-03-21 12:11:37 PDT
Comment on attachment 13745 [details] proposed fix r=me I think according to the style guidelines the outer ifs should have braces.
Alexey Proskuryakov
Comment 7 2007-03-21 12:25:30 PDT
Committed revision 20363. I do not know if it's correct for d="" to emit an error message, but the crasher is fixed now.
Note You need to log in before you can comment on or make changes to this bug.