Bug 18137

Summary: Crash on acid3 test 54
Product: WebKit Reporter: kappasolaris <webkit.20.kappa>
Component: DOMAssignee: Nobody <webkit-unassigned>
Status: RESOLVED WONTFIX    
Severity: Normal CC: ahmad.saleem792, ap, bfulgham, mrowe, rniwa
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Other   
Attachments:
Description Flags
Crash dump none

kappasolaris
Reported 2008-03-27 02:24:40 PDT
I'm getting a crash on acid3 test 54, and apparently changeset 31273 is at fault here, specifically this change in dom/Element.cpp: - String localName = shouldIgnoreAttributeCase(this) ? name.lower() : name; + const String& localName = (shouldIgnoreAttributeCase(this) && !name.impl()->isLower()) ? name.lower() : name; Note the addition of "&", if I remove it, it's all good again...
Attachments
Crash dump (4.42 KB, text/plain)
2008-03-27 02:26 PDT, kappasolaris 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
kappasolaris
Comment 1 2008-03-27 02:26:24 PDT
Created attachment 20120 [details] Crash dump
kappasolaris
Comment 2 2008-03-27 02:42:22 PDT
To clarify, I don't know if it's specifically test 54, I just go to http://acid3.acidtests.org/ and it crashes while showing 54/100.
kappasolaris
Comment 3 2008-03-27 02:49:52 PDT
That's on OpenSolaris with Sun Studio 12 compiler.
Mark Rowe (bdash)
Comment 4 2008-03-27 23:55:07 PDT
The fact that removing the & fixes the crash for you suggests that this may be a compiler optimisation bug.
Ahmad Saleem
Comment 5 2022-08-09 10:11:37 PDT
Solaris support was removed in 2017: https://bugs.webkit.org/show_bug.cgi?id=176341 Marking this as "RESOLVED WONTFIX". Thanks!
Note You need to log in before you can comment on or make changes to this bug.