Bug 8693

Summary:

REGRESSION: Crash occurs when reloading PDF file

Product:

WebKit

Reporter:

Chris Petersen <c.petersen87>

Component:

Text

Assignee:

Nobody <webkit-unassigned>

Status:

VERIFIED FIXED

Severity:

Major

CC:

bdakin, mike.wessler, mitz

Priority:

Keywords:

InRadar, Regression

Version:

420+

Hardware:

Mac

OS:

OS X 10.4

URL:

http://www.uscis.gov/graphics/formsfee/forms/files/i-9.pdf

Attachments:

Description	Flags
Backtrace	none
Patch that fixes some problems (but still crash further along)	none
Patch to fix crash	bdakin: review+

Chris Petersen

Reported 2006-05-02 08:22:01 PDT

This crash only occurs when reloading a PDF file with TOT Webkit. This is a recent regression since the wasn't occuring with the r14124 NB. I can reproduce this 100% with the r14154 NB. Steps to reproduce: 1) With TOT Webkit, go to http://www.uscis.gov/graphics/formsfee/forms/files/i-9.pdf 2) After PDF loads, reload the page 3) Crash occurs

Attachments
Backtrace (20.28 KB, text/plain) 2006-05-02 08:26 PDT, Chris Petersen	no flags	Details
Patch that fixes some problems (but still crash further along) (1.19 KB, patch) 2006-05-02 16:53 PDT, Dave Hyatt	no flags	Details Formatted Diff Diff
Patch to fix crash (1.86 KB, patch) 2006-05-04 10:58 PDT, Dave Hyatt	bdakin: review+	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Chris Petersen

Comment 1 2006-05-02 08:26:54 PDT

Created attachment 8074 [details] Backtrace

Chris Petersen

Comment 2 2006-05-02 09:54:54 PDT

This is filed as <rdar://problem/4533778>

Beth Dakin

Comment 3 2006-05-02 10:23:03 PDT

Moving this to the text component since that is where the backtrace is.

Alexey Proskuryakov

Comment 4 2006-05-02 13:27:08 PDT

*** Bug 8701 has been marked as a duplicate of this bug. ***

Dave Hyatt

Comment 5 2006-05-02 13:43:31 PDT

Although this is probably some memory management issue, I'm pretty astonished by the backtrace. We're clearing our font caches just because we loaded a PDF file? That seems crazy (and makes me worry we're observing the wrong notification).

Dave Hyatt

Comment 6 2006-05-02 16:53:31 PDT

Created attachment 8088 [details] Patch that fixes some problems (but still crash further along) This fixes 2 crashes. There's a third crash even after fixing these two though.

Dave Hyatt

Comment 7 2006-05-04 10:58:06 PDT

Created attachment 8106 [details] Patch to fix crash

Beth Dakin

Comment 8 2006-05-04 10:59:43 PDT

Comment on attachment 8106 [details] Patch to fix crash r=me

Dave Hyatt

Comment 9 2006-05-04 14:28:32 PDT

Fixed.

Chris Petersen

Comment 10 2006-05-13 23:09:59 PDT

This issue is fixed but there is a remaining crash triggered by PDF viewing. http://bugzilla.opendarwin.org/show_bug.cgi?id=8893

Note You need to log in before you can comment on or make changes to this bug.