WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED INVALID
170541
[GTK] [2.14.5] WebKitGTK+ always loads system-wide installed plugins
https://bugs.webkit.org/show_bug.cgi?id=170541
Summary
[GTK] [2.14.5] WebKitGTK+ always loads system-wide installed plugins
Alberto Garcia
Reported
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
Attachments
Add attachment
proposed patch, testcase, etc.
Jérémy Lal
Comment 1
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)
Michael Catanzaro
Comment 2
2017-04-06 07:31:17 PDT
That patch looks sane to me. Now does it work? :)
Jérémy Lal
Comment 3
2017-04-06 08:29:42 PDT
It doesn't. I'll try something else later.
Adrian Perez
Comment 4
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?
Carlos Garcia Campos
Comment 5
2020-08-17 06:46:39 PDT
Plugins are no longer supported.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug