Closed
Bug 791189
Opened 12 years ago
Closed 12 years ago
[b2g-bluetooth] Disabling bluetooth in settings app crashes main process
Categories
(Core :: General, defect)
Tracking
()
People
(Reporter: cjones, Assigned: qdot)
References
Details
Attachments
(1 file)
This didn't used to happen, so I suspect without any evidence bluetooth.
Reporter | ||
Comment 1•12 years ago
|
||
Disabling airplane mode after it's already on doesn't crash the b2g process.
Reporter | ||
Comment 2•12 years ago
|
||
If I don't enter airplane mode, but just disable bluetooth, crash. Sorry :(.
Summary: Enabling airplane mode in OOP settings app crashes main process → Disabling bluetooth in OOP settings app crashes main process
Comment 3•12 years ago
|
||
This should become a basecamp-blocker, as we're about to run Settings Out Of Process: https://github.com/mozilla-b2g/gaia/pull/4728
blocking-basecamp: --- → ?
Hm. I have tested this a bunch on desktop, is this something unique to the phone maybe? I can't reproduce and there's no stack or anything to go on here...
Reporter | ||
Comment 5•12 years ago
|
||
Only tested on phone. Didn't have time for a stack yesterday.
blocking-basecamp: ? → +
Assignee | ||
Comment 6•12 years ago
|
||
Can confirm happening on phone and not on desktop, was happening before OOP landed too.
Assignee: nobody → kyle
Reporter | ||
Comment 7•12 years ago
|
||
Do you mean OOP'ing of bluetooth, or of the settings app?
Reporter | ||
Comment 8•12 years ago
|
||
My reading of comment 6 is that this is a preexisting condition for in-process bluetooth. Please correct if that's wrong.
Summary: Disabling bluetooth in OOP settings app crashes main process → Disabling bluetooth in settings app crashes main process
Reporter | ||
Updated•12 years ago
|
Summary: Disabling bluetooth in settings app crashes main process → [b2g-bluetooth] Disabling bluetooth in settings app crashes main process
Assignee | ||
Comment 9•12 years ago
|
||
Repro update: this doesn't happen when bluetooth is disabled from the pulldown menu. Just happens in the settings app.
Assignee | ||
Comment 10•12 years ago
|
||
Ok, multiple issues here:
- Whenever we get a dbus signal that we don't have any specific action for ("DefaultAdapterChanged", for instance), we just fall out the bottom of our if/else block but still try to distribute the signal, which we never actually stored anywhere.
- Even after that's fixed, we still crash. That's because there's a sDefaultAdapterPath variable that used to store the path of the Adapter after an AdapterAdded signal is sent. This is all well and good when the AdapterAdded signal is actually sent. However, say B2G is taken down and comes back up without rebooting. gecko will enable bluetooth, but it was never disabled, so nothing happens, no new AdapterAdded signal is sent, path is never filled in.
Assignee | ||
Comment 11•12 years ago
|
||
Ok. Reserved record service registration is straight out wrong. Removing for now to fix crasher, will file followup.
Assignee | ||
Comment 12•12 years ago
|
||
Attachment #661471 -
Flags: review?(dhylands)
Updated•12 years ago
|
Attachment #661471 -
Flags: review?(dhylands) → review+
Assignee | ||
Comment 13•12 years ago
|
||
Target Milestone: --- → mozilla18
Comment 14•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•