Closed Bug 539063 Opened 10 years ago Closed 10 years ago

Implement a whitelist and blacklist for OOPP

Categories

(Core :: Plug-ins, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
Tracking Status
status1.9.2 --- .4-fixed

People

(Reporter: benjamin, Assigned: jaas)

References

Details

(Whiteboard: [fixed-lorentz])

Attachments

(1 file, 1 obsolete file)

For OOPP, it's likely that we'll want to start out with a whitelist of plugins that we've tested work with OOPP. At some point we may want to flip it to a blacklist of plugins that are known not to work.

Josh, can you take this or find an owner? We probably want to do this with prefs, but I'm not sure whether we can reliably key off the plugin filename or some other marker.
Assignee: nobody → joshmoz
OS: Linux → All
Hardware: x86 → All
Attached patch fix v1.0 (obsolete) — Splinter Review
I've only tested this on Linux so far but it probably works fine on Windows.

You can specify per-library exceptions to whatever "dom.ipc.plugins.enabled" indicates by setting boolean prefs like "dom.ipc.plugins.enabled.libnpfoo.so".
Attachment #423592 - Flags: review?(benjamin)
Josh, when reviewing this I thought of one issue and cleaned it up so we didn't need to use the localfile:

* because windows isn't case-sensitive, I've normalized the plugin name to lowercase before checking the pref
* Use string function RFindCharInSet instead of localfile->GetNativeLeafName because it's quicker and cleaner
Attachment #423854 - Flags: review?(joshmoz)
Attachment #423592 - Attachment is obsolete: true
Attachment #423592 - Flags: review?(benjamin)
Attachment #423854 - Flags: review?(joshmoz) → review+
http://hg.mozilla.org/mozilla-central/rev/ac98eb7edabc
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
How exactly do I use this?  I have a plugin that doesn't play well with OOPP and I'd like to blacklist it.

Thanks
Starting with tomorrow's nightly:
* find the file name of your plugin by looking in about:plugins
* go to about:config
* create a new boolean pref "dom.ipc.plugins.enabled.<filename>" with value "false"

BTW, what plugin are you blacklisting?
Foxit Reader, the latest version.
looks like Bug 542792
Would it be possible to specify a leafname without the extension so for example
dom.ipc.plugins.enabled.npswf32

will allow NPSWF32.dll NPSWF32.so NPSWF32.dylib etc.
The Flash plugin is:
npswf32.dll on Windows
libflashplayer.so on Linux
"Flash Player.plugin" on OS X.

I don't see how that would help anything.
Blanket approval for Lorentz merge to mozilla-1.9.2
a=beltzner for 1.9.2.4 - please make sure to mark status1.9.2:.4-fixed
While this still remains a whitelist, what mechanism is in place to add plugins to the whitelist by default in newer releases?
If no bug about that exists, should one be filed based on stats gained from http://www.mozilla.com/en-US/plugincheck/ showing a rank of the most installed plugins?
You need to log in before you can comment on or make changes to this bug.