Bug 34443

Summary: [GTK] Crash on testwebbackforwardlist unit test
Product: WebKit Reporter: Gustavo Noronha (kov) <gustavo>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: beidson, fishd
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
Stack trace of the crash
none
Add missing null-check beidson: review-, gustavo: commit-queue-

Gustavo Noronha (kov)
Reported 2010-02-01 12:15:09 PST
Created attachment 47861 [details] Stack trace of the crash This is crashing since r53950 - reverting that revision fixes the crash. Trace attached.
Attachments
Stack trace of the crash (4.01 KB, text/plain)
2010-02-01 12:15 PST, Gustavo Noronha (kov) 		no flags
Details
Add missing null-check (2.13 KB, patch)
2010-02-01 12:19 PST, Gustavo Noronha (kov) 		beidson: review-
gustavo: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Gustavo Noronha (kov)
Comment 1 2010-02-01 12:19:20 PST
Created attachment 47862 [details] Add missing null-check
Brady Eidson
Comment 2 2010-02-01 12:33:09 PST
Comment on attachment 47862 [details] Add missing null-check Please hold off on this - we need a more complete fix. (This is not GTK-only) See https://bugs.webkit.org/show_bug.cgi?id=34444
Brady Eidson
Comment 3 2010-02-01 12:33:58 PST
Reverse duping because 34444 has a more complete fix. *** This bug has been marked as a duplicate of bug 34444 ***
Brady Eidson
Comment 4 2010-02-01 12:35:13 PST
Also, sorry for the reverse dupe. I'd actually been working on this crash for the last few hours (oh what fun adding DRT changes and tests for specific API usage...) and filed 34444 before I saw this existed. What a difference 10 minutes makes.
Gustavo Noronha (kov)
Comment 5 2010-02-01 13:24:51 PST
(In reply to comment #4) > Also, sorry for the reverse dupe. I'd actually been working on this crash for > the last few hours (oh what fun adding DRT changes and tests for specific API > usage...) and filed 34444 before I saw this existed. What a difference 10 > minutes makes. Nevermind, I made a simple fix because this would hold up our release, I'm happy to see a much more solid fix landing, thanks!
Note You need to log in before you can comment on or make changes to this bug.