The default bug view has changed. See this FAQ.

Making a geolocation permission crashes on html4demos.com/geo

VERIFIED FIXED in mozilla17

Status

()

Core
DOM: Device Interfaces
--
critical
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: tchung, Assigned: kanru)

Tracking

Trunk
mozilla17
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(blocking-basecamp:+)

Details

(URL)

Attachments

(2 attachments)

(Reporter)

Description

5 years ago
Created attachment 645980 [details]
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

Updated

5 years ago
Summary: Making a geolocation data call crashes on html4demos.com/geo → Making a geolocation permission crashes on html4demos.com/geo
(Assignee)

Comment 1

5 years ago
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"));
(Assignee)

Comment 2

5 years ago
Created attachment 646226 [details] [diff] [review]
Add more check to GonkGPSGeolocationProvider
Assignee: nobody → kchen
Attachment #646226 - Flags: review?(doug.turner)

Updated

5 years ago
Attachment #646226 - Flags: review?(doug.turner) → review+
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
(Assignee)

Comment 3

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/d61eadd4c511
Target Milestone: --- → mozilla17
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/d61eadd4c511
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
blocking-basecamp: ? → +

Comment 5

5 years ago
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

5 years ago
2012-07-31 #VERIFIED
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.