Bug 32600

Summary: [Qt] fast/css/font-face-unused-source-loaded.html crashes
Product: WebKit Reporter: Csaba Osztrogonác <ossy>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: hausmann, webkit.review.bot
Priority: P3 Keywords: Qt
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
Patch
none
Patch zecke: review+

Description Csaba Osztrogonác 2009-12-16 00:44:33 PST 
fast/css/font-face-unused-source-loaded.html introduced by 
http://trac.webkit.org/changeset/52174 crashed with QtWebKit on Linux.
( original bug: https://bugs.webkit.org/show_bug.cgi?id=32257 )

crash caused by the test before: fast/css/font-face-multiple-families.html

stderr on buildbot:
http://build.webkit.org/results/Qt%20Linux%20Release/r52188%20%284974%29/fast/css/font-face-unused-source-loaded-stderr.txt

In debug mode DumpRenderTree said there is an stderr file, but it isn't exist unfortunately.
Comment 1 Csaba Osztrogonác 2009-12-16 01:03:54 PST 
fast/css/font-face-multiple-families.html added to skiplist until fix. 
http://trac.webkit.org/changeset/52190 (to keep buildbot green)
Comment 2 Simon Hausmann 2009-12-16 01:25:46 PST 
Seems this line

font.setFamily(QFontDatabase::applicationFontFamilies(m_handle)[0]);

in FontCustomPlatformData::fontPlatformData causes it (out of bounds array access).
Comment 3 Simon Hausmann 2009-12-16 01:50:54 PST 
Created attachment 44958 [details]
Patch
Comment 4 WebKit Review Bot 2009-12-16 01:52:05 PST 
style-queue ran check-webkit-style on attachment 44958 [details] without any errors.
Comment 5 Simon Hausmann 2009-12-16 01:56:07 PST 
Created attachment 44959 [details]
Patch
Comment 6 WebKit Review Bot 2009-12-16 01:57:25 PST 
style-queue ran check-webkit-style on attachment 44959 [details] without any errors.
Comment 7 Holger Freyther 2009-12-16 01:59:59 PST 
Comment on attachment 44959 [details]
Patch

I'm so incredible happy that "we" address regressions in a timely manner!
Comment 8 Simon Hausmann 2009-12-16 02:07:21 PST 
Committed r52194: <http://trac.webkit.org/changeset/52194>