5541 2005-10-29 02:25:37 -0700 REGRESSION: crash on importing bookmarks 2005-11-01 13:42:15 -0800 1 1 1 Unclassified WebKit WebKit Misc. 420+ Mac OS X 10.4 VERIFIED FIXED P1 Normal --- 1 ap mjs jon oldest_to_newest 23158 0 ap 2005-10-29 02:25:37 -0700 HTMLNames constants are not yet initialized at Safari startup, when bookmarks are imported, which surely causes a crash as soon as they are used (which happens to be in Decoder::decode()). Steps to reproduce: 1. Launch MSIE at least once, to make it write out its bookmarks 2. defaults write com.apple.Safari IEFavoritesWereImported 0 3. run-safari Results: Safari crashes Crash log (as reported by erickt on IRC): #0 0x019432a8 in DOM::AtomicString::equal (a=@0xbfffeae0, b=@0x8) at /unencrypted/erickt/ Projects/WebKit/WebCore/khtml/xml/dom_atomicstring.h:80 #1 0x016a51b4 in khtml::Decoder::decode (this=0x6999b0, data=0xf02e000 "<HTML>\r<!DOCTYPE NETSCAPE-Bookmark-file-1>\r<!Created by Microsoft Internet Explorer 4.0 >\r<META HTTP-EQUIV= \"Content-Type\" CONTENT=\"text/html; charset=x-mac-roman\">\r<TITLE>Favorites</TITLE> \r<H1 WINDOW_"..., len=1092) at /unencrypted/erickt/Projects/WebKit/WebCore/khtml/misc/ decoder.cpp:556 #2 0x01709c1c in +[WebCoreEncodings decodeData:] (self=0xbfffeae0, _cmd=0x8, data=0xec32da0) at /unencrypted/erickt/Projects/WebKit/WebCore/kwq/WebCoreEncodings.mm:36 #3 0x0009d238 in ?? () #4 0x0005b224 in ?? () #5 0x0005b2bc in ?? () #6 0x00005d54 in ?? () #7 0x00002d38 in ?? () #8 0x90b05408 in -[NSSet makeObjectsPerformSelector:] () #9 0x9f664f94 in -[NSIBObjectData nibInstantiateWithOwner:topLevelObjects:] () #10 0x9f73bbdc in old_loadNib () #11 0x9f650df0 in +[NSBundle(NSNibLoading) _loadNibFile:nameTable:withZone:ownerBundle:] () #12 0x9f6a805c in +[NSBundle(NSNibLoading) loadNibFile:externalNameTable:withZone:] () #13 0x9f737e10 in +[NSBundle(NSNibLoading) loadNibNamed:owner:] () #14 0x9f737bb0 in NSApplicationMain () #15 0x000021e8 in ?? () #16 0x00056e28 in ?? () 23159 1 4518 ap 2005-10-29 02:40:56 -0700 Created attachment 4518 proposed patch 23415 2 darin 2005-10-31 19:45:17 -0800 *** Bug 4351 has been marked as a duplicate of this bug. *** 23443 3 timothy 2005-11-01 09:34:36 -0800 Landed this. Checking in kwq/WebCoreEncodings.mm; /cvs/root/WebCore/kwq/WebCoreEncodings.mm,v <-- WebCoreEncodings.mm new revision: 1.8; previous revision: 1.7 4518 2005-10-29 02:40:56 -0700 2005-10-29 02:51:07 -0700 proposed patch InitHTMLNamesPatch.txt text/plain 783 ap SW5kZXg6IFdlYkNvcmVFbmNvZGluZ3MubW0KPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQpSQ1MgZmlsZTogL2N2cy9yb290 L1dlYkNvcmUva3dxL1dlYkNvcmVFbmNvZGluZ3MubW0sdgpyZXRyaWV2aW5nIHJldmlzaW9uIDEu NwpkaWZmIC1wIC11IC1yMS43IFdlYkNvcmVFbmNvZGluZ3MubW0KLS0tIFdlYkNvcmVFbmNvZGlu Z3MubW0JMyBPY3QgMjAwNSAyMToxMzoxMiAtMDAwMAkxLjcKKysrIFdlYkNvcmVFbmNvZGluZ3Mu bW0JMjkgT2N0IDIwMDUgMDk6NDA6MTkgLTAwMDAKQEAgLTI4LDEwICsyOCwxMyBAQAogCiAjaW1w b3J0ICJkZWNvZGVyLmgiCiAKKyNpbmNsdWRlICJodG1sbmFtZXMuaCIKKwogQGltcGxlbWVudGF0 aW9uIFdlYkNvcmVFbmNvZGluZ3MKIAogKyAoTlNTdHJpbmcgKilkZWNvZGVEYXRhOihOU0RhdGEg KilkYXRhCiB7CisgICAgRE9NOjpIVE1MTmFtZXM6OmluaXQoKTsgLy8gdGhpcyBtZXRob2QgaXMg dXNlZCBmb3IgaW1wb3J0aW5nIGJvb2ttYXJrcyBhdCBzdGFydHVwLCBzbyBIVE1MTmFtZXMgYXJl IGxpa2VseSB0byBiZSB1bmluaXRpYWxpemVkIHlldAogICAgIGtodG1sOjpEZWNvZGVyICpkZWNv ZGVyID0gbmV3IGtodG1sOjpEZWNvZGVyKCk7CiAgICAgUVN0cmluZyByZXN1bHQgPSBkZWNvZGVy LT5kZWNvZGUoc3RhdGljX2Nhc3Q8Y29uc3QgY2hhciAqPihbZGF0YSBieXRlc10pLCBbZGF0YSBs ZW5ndGhdKTsKICAgICByZXN1bHQgKz0gZGVjb2Rlci0+Zmx1c2goKTsK