Closed
Bug 784766
Opened 12 years ago
Closed 12 years ago
Wifi: Verify that managers that talk to us have permission to do so
Categories
(Core :: DOM: Device Interfaces, defect)
Core
DOM: Device Interfaces
Tracking
()
People
(Reporter: mrbkap, Assigned: mrbkap)
References
Details
(Whiteboard: [LOE:S] [WebAPI:P0])
Attachments
(1 file)
653 bytes,
patch
|
cjones
:
review+
mrbkap
:
checkin+
|
Details | Diff | Splinter Review |
See bug 777203 comment 3.
Updated•12 years ago
|
Whiteboard: [LOE:S] → [LOE:S] [WebAPI:P0]
Comment 2•12 years ago
|
||
Blake, can you provide an update here?
Assignee | ||
Comment 3•12 years ago
|
||
bug 776832 adds the ability to fix this bug. Once it's checked, in I'll patch this right away.
Comment 4•12 years ago
|
||
Ping, looks like the dependent bug has been fixed.
Assignee | ||
Comment 5•12 years ago
|
||
I verified that this worked by removing the "wifi-manage" permission from the settings app and hacking the DOM side to not do any security checks. The settings app was killed as expected.
Attachment #667012 -
Flags: review?(jones.chris.g)
Updated•12 years ago
|
Attachment #667012 -
Flags: review?(jones.chris.g) → review+
Assignee | ||
Comment 6•12 years ago
|
||
Comment on attachment 667012 [details] [diff] [review] Proposed fix https://hg.mozilla.org/integration/mozilla-inbound/rev/6ec6bc05c62f
Attachment #667012 -
Flags: checkin+
Comment 7•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/6ec6bc05c62f
Status: NEW → RESOLVED
Closed: 12 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
Target Milestone: --- → mozilla18
You need to log in
before you can comment on or make changes to this bug.
Description
•