Bug 16375 - REGRESSION: Safari crashes on quit
Summary: REGRESSION: Safari crashes on quit
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.4
: P1 Normal
Assignee: Darin Adler
URL:
Keywords: Regression
Depends on:
Blocks:
 
Reported: 2007-12-09 23:00 PST by mitz
Modified: 2007-12-10 05:15 PST (History)
1 user (show)

See Also:


Attachments
patch (834 bytes, patch)
2007-12-09 23:24 PST, Darin Adler
no flags Details | Formatted Diff | Diff
patch (this time with the correct ChangeLog) (1.05 KB, patch)
2007-12-09 23:28 PST, Darin Adler
mrowe: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description mitz 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
Comment 1 Darin Adler 2007-12-09 23:22:35 PST
What a coincidence! I have a patch for this already.
Comment 2 Darin Adler 2007-12-09 23:24:28 PST
Created attachment 17814 [details]
patch
Comment 3 Darin Adler 2007-12-09 23:28:32 PST
Created attachment 17815 [details]
patch (this time with the correct ChangeLog)
Comment 4 Mark Rowe (bdash) 2007-12-10 05:15:34 PST
I landed this in r28580 as it was causing havoc on the buildbots.