Security/Reviews/FxOSGecko/Template: Difference between revisions

From MozillaWiki
Jump to navigation Jump to search
No edit summary
Line 22: Line 22:
** Messaging (child/parent)  
** Messaging (child/parent)  


The following items were deemed lower risk and not reviewed:
As the API is still in development, this review is a design review only (testing to be conducted at a later stage).


===Components===
===Components===

Revision as of 02:21, 26 September 2014

WORK IN PROGRESS

FirefoxOS Review Details

  • API: Bluetooth API - Version 2
  • Review Date: September 2014
  • Review Lead: Paul Theriault

Context

Work is currently underway to implement a new version of the Gecko Bluetooth API. The key changes are:

  • new API, using webidl
  • Bluetooth will be exposed to privileged apps
  • new bluetooth profiles supported

Scope

This review focuses specifically on the new API itself. Review of the Gaia bluetooth app Configuration of Wifi via the settings (and other Apps)

As the API is still in development, this review is a design review only (testing to be conducted at a later stage).

Components

Relevant Source Code

The code for the new API is stored in the /dom/bluetooth2 directory of gecko source code: http://dxr.mozilla.org/mozilla-central/source/dom/bluetooth2

Permission Model

  • Paste from Permissions Table.jsm (see below)
  • Discuss anything special like access
  • Discuss where permissions are enforced (access to object, on IPC messages, at each function call etc)
 "wifi-manage": {
 190                              app: DENY_ACTION,
 191                              privileged: DENY_ACTION,
 192                              certified: ALLOW_ACTION
 193                            },

Review Notes

1. Content/Chrome Segregation

2. Process Segregation

3. Data validation & Sanitization

4. Denial of Service

Security Risks & Mitigating Controls

Actions & Recommendations

  • List of recommendations, and corresponding bug numbers
  • For sensitive bugs, just put bug number (or omit entirely maybe it is really dangerous & obvious)