6816 2006-01-25 20:12:41 -0800 Crash in dispatchMouseEvent - Missing nil check 2006-01-25 20:34:22 -0800 1 1 1 Unclassified WebKit New Bugs 420+ Mac OS X 10.4 RESOLVED FIXED P2 Major --- 1 adele adele oldest_to_newest 29967 0 adele 2006-01-25 20:12:41 -0800 Date/Time: 2006-01-25 20:04:22.271 -0800 OS Version: 10.4.4 (Build 8G32) Report Version: 3 Command: Safari Path: /Build/symroots/Development/Safari.app/Contents/MacOS/Safari Parent: bash [19431] Version: 2.0.1 (420+) Build Version: 1 Project Name: WebBrowser Source Version: 4161300 PID: 22324 Thread: 0 Exception: EXC_BAD_ACCESS (0x0001) Codes: KERN_PROTECTION_FAILURE (0x0002) at 0x00000018 Thread 0 Crashed: 0 com.apple.WebCore 0x020b7a6c WebCore::NodeImpl::renderer() const + 20 (NodeImpl.h:336) 1 com.apple.WebCore 0x01f00248 FrameView::dispatchMouseEvent(WebCore::AtomicString const&, WebCore::NodeImpl*, bool, int, QMouseEvent*, bool, int) + 1204 (FrameView.cpp:1089) 2 com.apple.WebCore 0x01f010dc FrameView::viewportMousePressEvent(QMouseEvent*) + 644 (FrameView.cpp:544) 3 com.apple.WebCore 0x01ef3b08 MacFrame::mouseDown(NSEvent*) + 732 (MacFrame.mm:2085) 4 com.apple.WebCore 0x01f280b8 -[WebCoreFrameBridge mouseDown:] + 52 (WebCoreFrameBridge.mm:1106) 5 com.apple.WebKit 0x0127dc38 -[WebHTMLView mouseDown:] + 492 (WebHTMLView.m:2653) 6 com.apple.AppKit 0x936d5eb0 -[NSWindow sendEvent:] + 4616 7 com.apple.Safari 0x00100200 -[Window sendEvent:] + 320 (Window.m:76) 8 com.apple.AppKit 0x9367eef4 -[NSApplication sendEvent:] + 4172 9 com.apple.Safari 0x00030910 -[BrowserApplication sendEvent:] + 736 (BrowserApplication.m:139) 10 com.apple.AppKit 0x93676330 -[NSApplication run] + 508 11 com.apple.AppKit 0x93766e68 NSApplicationMain + 452 12 com.apple.Safari 0x000f8c3c main + 160 (main.m:23) 13 com.apple.Safari 0x00002510 _start + 340 (crt.c:272) 14 com.apple.Safari 0x000023b8 start + 60 29968 1 adele 2006-01-25 20:14:13 -0800 I'm not entirely sure how to reproduce this, but it was caused by my recent change to support the new text fields. I have a fix... 29969 2 5968 adele 2006-01-25 20:17:52 -0800 Created attachment 5968 patch 29970 3 5968 mjs 2006-01-25 20:30:43 -0800 Comment on attachment 5968 patch You could consider using ?: syntax instead if if() to do the conditional initialization. But fine to commit either way. 5968 2006-01-25 20:17:52 -0800 2006-01-25 20:30:43 -0800 patch patch3.txt text/plain 1253 adele SW5kZXg6IENoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBDaGFuZ2VMb2cJKHJldmlzaW9uIDEyMzc5 KQorKysgQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTMgQEAKKzIwMDYtMDEt MjUgIEFkZWxlIFBldGVyc29uICA8YWRlbGVAYXBwbGUuY29tPgorCisgICAgICAgIFJldmlld2Vk IGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIGh0dHA6Ly9idWd6aWxsYS5vcGVuZGFyd2lu Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NjgxNgorICAgICAgICBDcmFzaCBpbiBkaXNwYXRjaE1vdXNl RXZlbnQgLSBNaXNzaW5nIG5pbCBjaGVjaworCisgICAgICAgICogcGFnZS9GcmFtZVZpZXcuY3Bw OgorICAgICAgICAoRnJhbWVWaWV3OjpkaXNwYXRjaE1vdXNlRXZlbnQpOgorCiAyMDA2LTAxLTI1 ICBKdXN0aW4gR2FyY2lhICA8anVzdGluLmdhcmNpYUBhcHBsZS5jb20+CiAKICAgICAgICAgUmV2 aWV3ZWQgYnkgbWpzCkluZGV4OiBwYWdlL0ZyYW1lVmlldy5jcHAKPT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gcGFn ZS9GcmFtZVZpZXcuY3BwCShyZXZpc2lvbiAxMjM0OSkKKysrIHBhZ2UvRnJhbWVWaWV3LmNwcAko d29ya2luZyBjb3B5KQpAQCAtMTA4Niw3ICsxMDg2LDkgQEAKICAgICAgICAgLy8gaXMgZXhwZWN0 ZWQgYnkgc29tZSBzaXRlcyB0aGF0IHJlbHkgb24gb25DaGFuZ2UgaGFuZGxlcnMgcnVubmluZwog ICAgICAgICAvLyBmcm9tIGZvcm0gZmllbGRzIGJlZm9yZSB0aGUgYnV0dG9uIGNsaWNrIGlzIHBy b2Nlc3NlZC4KICAgICAgICAgTm9kZUltcGwqIG5vZGUgPSB0YXJnZXROb2RlOwotICAgICAgICBS ZW5kZXJPYmplY3QqIHJlbmRlcmVyID0gbm9kZS0+cmVuZGVyZXIoKTsKKyAgICAgICAgUmVuZGVy T2JqZWN0KiByZW5kZXJlciA9IDA7CisgICAgICAgIGlmIChub2RlKQorICAgICAgICAgICAgcmVu ZGVyZXIgPSBub2RlLT5yZW5kZXJlcigpOwogICAgICAgICAgICAgICAgIAogICAgICAgICAvLyBX YWxrIHVwIHRoZSByZW5kZXIgdHJlZSB0byBzZWFyY2ggZm9yIGEgbm9kZSB0byBmb2N1cy4KICAg ICAgICAgLy8gV2Fsa2luZyB1cCB0aGUgRE9NIHRyZWUgd291bGRuJ3Qgd29yayBmb3Igc2hhZG93 IHRyZWVzLCBsaWtlIHRob3NlIGJlaGluZCB0aGUgZW5naW5lLWJhc2VkIHRleHQgZmllbGRzLgo=