Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Privacy Badger has crashed Click this balloon to reload the extension #1047

Closed
m0lz opened this issue Dec 13, 2016 · 9 comments
Closed

Privacy Badger has crashed Click this balloon to reload the extension #1047

m0lz opened this issue Dec 13, 2016 · 9 comments
Assignees

Comments

@m0lz
Copy link

m0lz commented Dec 13, 2016

Privacy Badger Version 2016.12.8

Browser = http://chromium.woolyss.com/
Nik wpo cfg all-codecs stable 64-bit
No sync • No WebRTC • No Widevine • 55.0.2883.87 (423768) • Friday, 9 Dec 2016

Problem = Every time I click the Baloon to reload Privacy Badger it crashes again and again and again

I have uninstalled it in case of file corruption, then re-installed the latest version again .. No joy the same problem immediately starts again.

@alexristich alexristich self-assigned this Dec 13, 2016
@groovecoder
Copy link
Contributor

groovecoder commented Dec 13, 2016

It looks like this version is built without WebRTC support which causes this.

@cooperq cooperq added the Chrome label Dec 13, 2016
@alexristich
Copy link
Contributor

I gave this a quick try on version 55.0.2883.87 and it seems to work out of the box. It seems the most likely scenario is related to WebRTC being disabled, as @groovecoder notes above. Privacy Badger has built-in protections against WebRTC IP address leakage. There are four modes in the WebRTC specification (#969 (comment)); Privacy Badger uses the 3rd one as the default, and if a user enables goes to the Options page and enables the setting to prevent IP address leakage it will enable the 4th and most private mode.

Would you be willing to enable WebRTC and try again to see if you experience the same issue?

@qbit
Copy link

qbit commented Dec 15, 2016

uBlock origin had a similar issue to this. They fixed it by adding a check like this: https://github.com/gorhill/uBlock/blob/master/platform/chromium/is-webrtc-supported.js

@mauricecaaaaaben
Copy link

I also have this issue and of course don't have WebRTC.

@qbit
Copy link

qbit commented Dec 18, 2016

I should also mention, some people are unable to enable webrtc - OpenBSD for example.

qbit added a commit to qbit/privacybadger that referenced this issue Dec 18, 2016
This resolves EFForg#1047 for me, but I don't have a good setup to test
other (webrtc enabled) boxes.

This check is similar to what uBlock Origin does.
qbit added a commit to qbit/privacybadger that referenced this issue Dec 20, 2016
This resolves EFForg#1047 for me, but I don't have a good setup to test
other (webrtc enabled) boxes.

This check is similar to what uBlock Origin does.

fix lint

fix constructor
@jhamberg
Copy link
Contributor

jhamberg commented Dec 22, 2016

This issue can be replicated by running Chromium build 57.0.2959.0 (64-bit) with WebRTC functionalities disabled. The crash is caused by the method cpn.webRTCIPHandlingPolicy.get in background.js#L183 and can be mitigated by checking for support when enabling WebRTC protection or checking its state. I will now create a pull request.

@qbit
Copy link

qbit commented Dec 22, 2016

@jhamberg this is from my OpenBSD box. WebRTC is disabled at build time (enable_webrtc=false):

cpn.webRTCIPHandlingPolicy.get
function (details, callback) {
    var getSchema = getFunctionParameters('get');
    validate([details, callback], getSchema);
    return sendRequest('types.ChromeSetting.get',

I have created PR #1097 for this issue already.

@jhamberg
Copy link
Contributor

jhamberg commented Dec 22, 2016

@qbit Your solution seems fine and successfully prevents the extension from crashing on load. There is also an issue with WebRTC setting in the options menu. There are two more methods which need this check isWebRTCIPProtectionEnabled in background.js#L583 and toggleWebRTCIPProtection in options.js#L472.

WebRTC availability should probably be checked and flagged when creating the badger object.

@m0lz
Copy link
Author

m0lz commented Jan 1, 2017

@jhamberg thanks for the fix, I wondered why I could no longer use PB on the Chromium forks here http://chromium.woolyss.com/ ( typically I use the Stable Installer • Archive • No sync • No WebRTC • No Widevine )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants