Bug 22454

Summary: REGRESSION (3.2-TOT): Crash below FontFallbackList::fontDataAt on jacobian.org
Product: WebKit Reporter: Johan Bergström <bugs>
Component: WebCore Misc.Assignee: mitz
Status: RESOLVED FIXED    
Severity: Normal CC: jacob, john.engelhart, mitz, mrowe
Priority: P1 Keywords: InRadar, NeedsReduction, Regression
Version: 528+ (Nightly build)   
Hardware: Mac (Intel)   
OS: OS X 10.5   
URL: http://jacobian.org/writing/merquery/
Attachments:
Description Flags
crash log
none
Avoid a style recalc while font tables are in an inconsistent state ap: review+

Johan Bergström
Reported 2008-11-24 06:11:45 PST
Webkit r38699 on Leopard 10.5.5 crashes when visiting mentioned web page. Crash log attached.
Attachments
crash log (28.61 KB, text/plain)
2008-11-24 06:12 PST, Johan Bergström 		no flags
Details
Avoid a style recalc while font tables are in an inconsistent state (5.88 KB, patch)
2008-11-27 15:30 PST, mitz 		ap: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Johan Bergström
Comment 1 2008-11-24 06:12:52 PST
Created attachment 25425 [details] crash log
Mark Rowe (bdash)
Comment 2 2008-11-24 06:14:50 PST
This doesn't crash for me with the same build. The crash is happening at FontFallbackList.cpp:103, when calling FontData::isLoading. It appears that the pure-virtual version of the function is being invoked, resulting in a jump to 0x0. It's not clear how this can happen.
mitz
Comment 3 2008-11-27 11:22:05 PST
<rdar://problem/6405550>
mitz
Comment 4 2008-11-27 15:30:24 PST
Created attachment 25561 [details] Avoid a style recalc while font tables are in an inconsistent state
mitz
Comment 5 2008-11-29 13:21:53 PST
*** Bug 22406 has been marked as a duplicate of this bug. ***
Alexey Proskuryakov
Comment 6 2008-11-29 13:32:38 PST
Comment on attachment 25561 [details] Avoid a style recalc while font tables are in an inconsistent state r=me, but please consider adding some comments, as discussed on IRC.
mitz
Comment 7 2008-11-29 13:40:59 PST
Fixed in r38843 <http://trac.webkit.org/changeset/38843>.
Johan Bergström
Comment 8 2008-12-01 13:17:46 PST
If it should matter - works for me, thanks.
Yuzo Fujishima
Comment 9 2010-09-12 22:31:30 PDT
*** Bug 21467 has been marked as a duplicate of this bug. ***
Note You need to log in before you can comment on or make changes to this bug.