777597 - Making a geolocation permission crashes on html4demos.com/geo

Status: VERIFIED FIXED

(Core :: DOM: Device Interfaces, defect)

Description

[Tony Chung [:tchung]]

Attachment: logcat of geo crash

When trying to accept geolocation permissions, browser App will crash and restart. 

logcat attached, but not sure if its helpful.  Also, bug 774580 may be related, and unsure if the patch there will fix this problem.

Environment
-07-25-2012 daily build
- gaia commit: 578593de74dd70a0aa6c3d4a5f652d739e342694
- gecko commit: 3632e0895ae248b818dd50317534b4cb4cdc934a

Repro:
1) install daily on otoro
2) launch browser App, and go to URL
3) click YES on the geolocation doorhanger
4) wait a bit, and verify the browser app will crash and restart to lockscreen

Expected;
- hitting yes to Geolocation should not crash the browser, and instead load the page 

Actual:
- crash and restart lockscreen

Comment 1

[Kan-Ru Chen [:kanru] (UTC+9)]

Program received signal SIGSEGV, Segmentation fault.
0x40913710 in GonkGPSGeolocationProvider::ReleaseDataConnection (this=<value optimized out>)
    at /home/kanru/zone2/mozilla/central/dom/system/gonk/GonkGPSGeolocationProvider.cpp:299
299       mRIL->DeactivateDataCall(mCid, NS_LITERAL_STRING("Close SUPL session"));

Comment 2

[Kan-Ru Chen [:kanru] (UTC+9)]

Attachment: Add more check to GonkGPSGeolocationProvider

Comment 3

[Kan-Ru Chen [:kanru] (UTC+9)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/d61eadd4c511

Comment 4

[(no longer active)]

https://hg.mozilla.org/mozilla-central/rev/d61eadd4c511

Comment 5

[John Shih]

The fixed status has been verified
There is no crach anymore after accept the permission
but still can't get the geolocation information
please see https://github.com/mozilla-b2g/gaia/issues/2815

Comment 6

[John Shih]

2012-07-31 #VERIFIED