14387 2007-06-25 08:33:12 -0700 [S60] Pointer event in a blank browser window cusses crash 2011-03-21 11:53:32 -0700 1 1 1 Unclassified WebKit New Bugs 523.x (Safari 3) S60 Emulator S60 3rd edition CLOSED FIXED PlatformOnly P2 Blocker --- 1 Sachin.Padma webkit-unassigned oldest_to_newest 6360 0 Sachin.Padma 2007-06-25 08:33:12 -0700 - Steps to reproduce: Open BrowserNG, Option->Goto address, fill a non-existed local address to the address field, e.g. file://c:/ anything.htm, press open button. After no gateway error message, a blank browser window is shown up. Click on the blank browser window - Actual outcome: Crash - Expected outcome: Should not crash - Other info: I watched following calling sequence before crash: CPointerEventHandler::HandlePointerEventL -> CWebCoreBridge::ActivateEvent -> KWQKHTMLPart::activateNodeAtPoint If taking a look at the last function, renderer() returns null when there is no html document. The return value of renderer() should be checked. void KWQKHTMLPart::activateNodeAtPoint(const QPoint &point) { // On ActivateEvent, clear the "don't submit form twice" data member. prepareForUserAction(); RenderObject::NodeInfo renderInfo(true, false); renderer()->layer()->hitTest(renderInfo, point.x(), point.y()); TSW ID: YDAI-73V998 6361 1 15222 Sachin.Padma 2007-06-25 08:37:47 -0700 Created attachment 15222 Null pointer check 6317 2 15222 zalan 2007-06-25 11:51:53 -0700 Comment on attachment 15222 Null pointer check r=me 6216 3 Sachin.Padma 2007-06-26 07:42:13 -0700 landed in r23792 76965 4 bradley.morrison 2008-04-09 11:39:40 -0700 Bulk closing of all s60 platform bugs. Sorry for the noise! 370761 5 joel.parks 2011-03-21 11:53:32 -0700 re-purposing InTSW keyword for use by QtWebkit team 15222 2007-06-25 08:37:47 -0700 2007-06-25 11:51:53 -0700 Null pointer check YDAI-73V998.patch text/plain 1160 Sachin.Padma SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nDQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09DQotLS0gV2ViQ29yZS9DaGFuZ2VM b2cJKHJldmlzaW9uIDIzNzY0KQ0KKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkp DQpAQCAtMSwzICsxLDEzIEBADQorc3BhZG1hLCByZXZpZXdlZCBieSB6YWxhbg0KKyBERVNDOiBb UzYwXSBQb2ludGVyIGV2ZW50IGluIGEgYmxhbmsgYnJvd3NlciB3aW5kb3cgY2F1c2VzIGNyYXNo DQorIGh0dHA6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE0Mzg3DQorIFRTVyBJ RCA6ICBZREFJLTczVjk5OCANCisNCisgICAgICAgIFdBUk5JTkc6IE5PIFRFU1QgQ0FTRVMgQURE RUQgT1IgQ0hBTkdFRA0KKw0KKyAgICAgICAgKiBrd3EvS1dRS0hUTUxQYXJ0LmNwcDoNCisgICAg ICAgIChLV1FLSFRNTFBhcnQ6OmFjdGl2YXRlTm9kZUF0UG9pbnQpOg0KKw0KIGJybW9ycmlzLCBy ZXZpZXdlZCBieSBub2JvZHkNCiAgREVTQzogcmVnZW4gV2ViQ29yZV9zZGtVLkRFRg0KIA0KSW5k ZXg6IFdlYkNvcmUva3dxL0tXUUtIVE1MUGFydC5jcHANCj09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0NCi0tLSBXZWJDb3Jl L2t3cS9LV1FLSFRNTFBhcnQuY3BwCShyZXZpc2lvbiAyMTkxMSkNCisrKyBXZWJDb3JlL2t3cS9L V1FLSFRNTFBhcnQuY3BwCSh3b3JraW5nIGNvcHkpDQpAQCAtMTkwMCw2ICsxOTAwLDEwIEBAIHZv aWQgS1dRS0hUTUxQYXJ0OjphY3RpdmF0ZU5vZGVBdFBvaW50KGMNCiAgICAgLy8gT24gQWN0aXZh dGVFdmVudCwgY2xlYXIgdGhlICJkb24ndCBzdWJtaXQgZm9ybSB0d2ljZSIgZGF0YSBtZW1iZXIu DQogICAgIHByZXBhcmVGb3JVc2VyQWN0aW9uKCk7DQogDQorICAgIGlmKCFyZW5kZXJlcigpIHx8 ICFyZW5kZXJlci0+bGF5ZXIoKSkNCisgICAgICAgIHsNCisgICAgICAgIHJldHVybjsNCisgICAg ICAgIH0NCiAgICAgUmVuZGVyT2JqZWN0OjpOb2RlSW5mbyByZW5kZXJJbmZvKHRydWUsIGZhbHNl KTsNCiAgICAgcmVuZGVyZXIoKS0+bGF5ZXIoKS0+aGl0VGVzdChyZW5kZXJJbmZvLCBwb2ludC54 KCksIHBvaW50LnkoKSk7DQogDQo=