Bug 13691

Summary: REGRESSION: Crash when starting Webkit with JavaScript disabled
Product: WebKit Reporter: Elliott Sprehn <esprehn>
Component: JavaScriptCoreAssignee: Eric Seidel (no email) <eric>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, mjs
Priority: P1 Keywords: Regression
Version: 523.x (Safari 3)   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
Crash Log 1
none
Crash Log 2
none
easy fix mjs: review+

Elliott Sprehn
Reported 2007-05-11 23:53:40 PDT
When starting Webkit with JavaScript disabled the browser crashes immediately. Reproducible in the latest nightly. (Fri May 11 22:54:47 GMT 2007, r21420).
Attachments
Crash Log 1 (21.78 KB, text/plain)
2007-05-11 23:54 PDT, Elliott Sprehn 		no flags
Details
Crash Log 2 (21.17 KB, text/plain)
2007-05-11 23:55 PDT, Elliott Sprehn 		no flags
Details
easy fix (1.18 KB, patch)
2007-05-12 01:55 PDT, Eric Seidel (no email) 		mjs: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Elliott Sprehn
Comment 1 2007-05-11 23:54:54 PDT
Created attachment 14505 [details] Crash Log 1 Sample Crash Log
Elliott Sprehn
Comment 2 2007-05-11 23:55:58 PDT
Created attachment 14506 [details] Crash Log 2 Another Sample Crash log
Elliott Sprehn
Comment 3 2007-05-11 23:56:50 PDT
Crash logs are slightly different, but the last few things on the trace are the same.
Alexey Proskuryakov
Comment 4 2007-05-12 00:42:45 PDT
Reproduced in a local debug build by trying to open <http://nightly.webkit.org>. Thread 0 Crashed: 0 com.apple.WebCore 0x01640d1c _ZNK3WTF6RefPtrIN3KJS17ScriptInterpreterEEcvMS3_KFPS2_vEEv + 40 (RefPtr.h:56) 1 com.apple.WebCore 0x01640da8 WebCore::KJSProxy::haveInterpreter() const + 44 (kjs_proxy.h:56) 2 com.apple.WebCore 0x010f87fc WebCore::Frame::cleanupPlatformScriptObjects() + 96 (FrameMac.mm:735) 3 com.apple.WebCore 0x010f524c WebCore::Frame::cleanupScriptObjects() + 32 (Frame.cpp:1135) 4 com.apple.WebCore 0x01498a7c WebCore::FrameLoader::clear(bool) + 604 (FrameLoader.cpp:790) 5 com.apple.WebCore 0x01498b2c WebCore::FrameLoader::begin(WebCore::KURL const&) + 40 (FrameLoader.cpp:841) 6 com.apple.WebCore 0x01499078 WebCore::FrameLoader::receivedFirstData() + 52 (FrameLoader.cpp:805) 7 com.apple.WebCore 0x014992d4 WebCore::FrameLoader::setEncoding(WebCore::String const&, bool) + 72 (FrameLoader.cpp:1587)
Eric Seidel (no email)
Comment 5 2007-05-12 01:55:02 PDT
Created attachment 14508 [details] easy fix
Maciej Stachowiak
Comment 6 2007-05-12 01:57:05 PDT
Comment on attachment 14508 [details] easy fix r=me
Eric Seidel (no email)
Comment 7 2007-05-12 01:58:13 PDT
r21427
Note You need to log in before you can comment on or make changes to this bug.