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

Map TAG= to bootstrap phase info #16

Merged
merged 3 commits into from Feb 2, 2018
Merged

Conversation

irykoon
Copy link
Contributor

@irykoon irykoon commented Feb 1, 2018

This implementation will prepare the bootstrap phase info for being translated.

It will also mitigate the vulnerability described in: http://forums.dds6qkxpwdeubwucdiaord2xgbbeyds25rbsgr73tbfpqpt4a6vjwsyd.onion/t/graphical-gui-whonix-setup-wizard-anon-connection-wizard-technical-discussion/650/540

However, I choose not to prevent the weird SUMMARY= from showing up when an attacker manipulates both TAG= and SUMMARY=. Because:

  1. The implementation will be consistent with the Tor launcher doc:

If Tor Launcher cannot map the TAG to a localized string, it displays the SUMMARY text instead otherwise, the SUMMARY field is not used).

  1. When such an attack is performed, we need to stay alert of it somehow.

What I am wondering is if such an alert should be exposed to daily user. Or should we simply keep it in a log?

This implementation will mitigate the vuneralbility described in: http://forums.dds6qkxpwdeubwucdiaord2xgbbeyds25rbsgr73tbfpqpt4a6vjwsyd.onion/t/graphical-gui-whonix-setup-wizard-anon-connection-wizard-technical-discussion/650/540

It also prepares the bootstrap phase info for being translated.

The Tor-launcher implementation of this can be found here: https://gitweb.torproject.org/tor-launcher.git/plain/README-BOOTSTRAP
When DisableNetwork set to 1, doing systmctl restart tor@defautl.service is not will not be enough to open /vat/run/tor/control.

To workaround this, we have to do systemctl reload tor@default.service.
@adrelanos adrelanos merged commit 717f313 into Kicksecure:master Feb 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants