As a security precaution, we have turned on the setting "Require API key authentication for API requests" for everyone. If this has broken something, please contact
Last Comment Bug 786714 - TCPSocket is exposing too many interfaces to content
: TCPSocket is exposing too many interfaces to content
Product: Core
Classification: Components
Component: DOM: Device Interfaces (show other bugs)
: unspecified
: x86_64 Linux
: -- normal (vote)
: mozilla21
Assigned To: Glenna
: Andrew Overholt [:overholt]
Depends on:
  Show dependency treegraph
Reported: 2012-08-29 09:42 PDT by Josh Matthews [:jdm]
Modified: 2013-02-05 07:31 PST (History)
5 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

patch (v1) (967 bytes, patch)
2013-02-03 15:36 PST, Glenna
jonas: review+
josh: feedback+
Details | Diff | Splinter Review

Description User image Josh Matthews [:jdm] 2012-08-29 09:42:09 PDT
TCPSocket's clasSInfo currently has the following:

>    interfaces: [
>      Ci.nsIDOMTCPSocket,
>      Ci.nsIDOMGlobalPropertyInitializer,
>      Ci.nsIObserver,
>      Ci.nsISupportsWeakReference
>    ],

This is unnecessary. We should only be including nsIDOMTCPSocket in that list.
Comment 1 User image Glenna 2013-02-03 15:28:30 PST
I'll take on this bug.
Comment 2 User image Glenna 2013-02-03 15:36:01 PST
Created attachment 709534 [details] [diff] [review]
patch (v1)
Comment 3 User image Josh Matthews [:jdm] 2013-02-04 07:29:41 PST
Comment on attachment 709534 [details] [diff] [review]
patch (v1)

Review of attachment 709534 [details] [diff] [review]:

Thanks Glenna!
Comment 4 User image Josh Matthews [:jdm] 2013-02-05 02:32:14 PST
Landed with the following modified commit message: "Bug 786714 - Remove unnecessary interfaces from TCPSocket's classinfo. r=sicking"
Comment 5 User image Ryan VanderMeulen [:RyanVM] 2013-02-05 07:31:27 PST

Note You need to log in before you can comment on or make changes to this bug.