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+

Csaba Osztrogonác
Reported 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.
Attachments
Patch (1.51 KB, patch)
2009-12-16 01:50 PST, Simon Hausmann 		no flags
DetailsFormatted DiffDiff
Patch (2.80 KB, patch)
2009-12-16 01:56 PST, Simon Hausmann 		zecke: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Csaba Osztrogonác
Comment 1 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)
Simon Hausmann
Comment 2 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).
Simon Hausmann
Comment 3 2009-12-16 01:50:54 PST
Created attachment 44958 [details] Patch
WebKit Review Bot
Comment 4 2009-12-16 01:52:05 PST
style-queue ran check-webkit-style on attachment 44958 [details] without any errors.
Simon Hausmann
Comment 5 2009-12-16 01:56:07 PST
Created attachment 44959 [details] Patch
WebKit Review Bot
Comment 6 2009-12-16 01:57:25 PST
style-queue ran check-webkit-style on attachment 44959 [details] without any errors.
Holger Freyther
Comment 7 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!
Simon Hausmann
Comment 8 2009-12-16 02:07:21 PST
Committed r52194: <http://trac.webkit.org/changeset/52194>
Note You need to log in before you can comment on or make changes to this bug.