Bug 16375

Summary: REGRESSION: Safari crashes on quit
Product: WebKit Reporter: mitz
Component: JavaScriptCoreAssignee: Darin Adler <darin>
Status: RESOLVED FIXED    
Severity: Normal CC: darin
Priority: P1 Keywords: Regression
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
patch
none
patch (this time with the correct ChangeLog) mrowe: review+

mitz
Reported 2007-12-09 23:00:17 PST
Steps to reproduce: 1) Open Safari with TOT WebKit 2) Go to javascript: 0; 3) Quit Safari Result: Safari crashed. Backtrace: 0 com.apple.JavaScriptCore 0x005d8568 KJS::UString::Rep::deref() + 74 (ustring.h:159) 1 com.apple.JavaScriptCore 0x005d85bd WTF::RefPtr<KJS::UString::Rep>::~RefPtr() + 31 (RefPtr.h:45) 2 com.apple.JavaScriptCore 0x005d85d1 WTF::RefPtr<KJS::UString::Rep>::~RefPtr() + 17 (RefPtr.h:45) 3 com.apple.JavaScriptCore 0x005d85e5 KJS::UString::~UString() + 17 (ustring.h:229) 4 com.apple.JavaScriptCore 0x005d85f9 KJS::UString::~UString() + 17 (ustring.h:229) 5 com.apple.JavaScriptCore 0x005dbb63 KJS::Parser::~Parser() + 31 (Parser.h:41) 6 com.apple.JavaScriptCore 0x005dbb81 KJS::Parser::~Parser() + 17 (Parser.h:41) 7 com.apple.JavaScriptCore 0x00585125 __tcf_6 + 27 (Parser.cpp:111) 8 libSystem.B.dylib 0x901c195c __cxa_finalize + 241 9 libSystem.B.dylib 0x901c1850 exit + 33 10 com.apple.AppKit 0x91231060 -[NSApplication terminate:] + 772 11 com.apple.AppKit 0x91151f94 -[NSApplication sendAction:to:from:] + 112
Attachments
patch (834 bytes, patch)
2007-12-09 23:24 PST, Darin Adler 		no flags
DetailsFormatted DiffDiff
patch (this time with the correct ChangeLog) (1.05 KB, patch)
2007-12-09 23:28 PST, Darin Adler 		mrowe: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Darin Adler
Comment 1 2007-12-09 23:22:35 PST
What a coincidence! I have a patch for this already.
Darin Adler
Comment 2 2007-12-09 23:24:28 PST
Created attachment 17814 [details] patch
Darin Adler
Comment 3 2007-12-09 23:28:32 PST
Created attachment 17815 [details] patch (this time with the correct ChangeLog)
Mark Rowe (bdash)
Comment 4 2007-12-10 05:15:34 PST
I landed this in r28580 as it was causing havoc on the buildbots.
Note You need to log in before you can comment on or make changes to this bug.