Bug 170541 - [GTK] [2.14.5] WebKitGTK+ always loads system-wide installed plugins
Summary: [GTK] [2.14.5] WebKitGTK+ always loads system-wide installed plugins
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-04-06 02:06 PDT by Alberto Garcia
Modified: 2018-05-02 13:49 PDT (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alberto Garcia 2017-04-06 02:06:25 PDT
One Debian user reports that WebKitGTK+ is always loading system-wide installed plugins, even if the "enable-plugins" setting is explicitly set to FALSE.

This is apparently causing crashes in some of this WebKitGTK+ apps.

You can see that with this command:

   strace /usr/lib/x86_64-linux-gnu/webkit2gtk-4.0/MiniBrowser --enable-plugins=FALSE <some-URL>

Original bug report here: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=859100
Comment 1 Jérémy Lal 2017-04-06 02:22:15 PDT
Hello,

if the pluginInfoStore is not preloaded elsewhere, the behavior i'm seeing might be just right there:

--- a/Source/WebKit2/UIProcess/WebPageProxy.cpp
+++ b/Source/WebKit2/UIProcess/WebPageProxy.cpp
@@ -1315,8 +1315,9 @@
 
 #if ENABLE(NETSCAPE_PLUGIN_API)
     String newMimeType = mimeType;
+    if (!m_preferences->pluginsEnabled()) return false;
     PluginModuleInfo plugin = m_process->processPool().pluginInfoStore().findPlugin(newMimeType, URL());
-    if (!plugin.path.isNull() && m_preferences->pluginsEnabled())
+    if (!plugin.path.isNull())
         return true;
 #endif // ENABLE(NETSCAPE_PLUGIN_API)
Comment 2 Michael Catanzaro 2017-04-06 07:31:17 PDT
That patch looks sane to me. Now does it work? :)
Comment 3 Jérémy Lal 2017-04-06 08:29:42 PDT
It doesn't. I'll try something else later.
Comment 4 Adrian Perez 2018-05-02 13:49:21 PDT
(In reply to Jérémy Lal from comment #3)
> It doesn't. I'll try something else later.

Hi Jérémy! By any chance, did you get to try something else in the end?