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

Re-add security features based on GCC plugins in 4.18+ hardened kernels #53369

Merged
merged 3 commits into from Jan 4, 2019

Conversation

delroth
Copy link
Contributor

@delroth delroth commented Jan 4, 2019

Motivation for this change

9236990 disabled GCC plugins in 4.18+ in NixOS hardened kernels. This is undesirable given that tons of nice PaX features are getting ported through these plugins -- for example, STACKLEAK in 4.20.

This PR attempts to re-enable plugins + adds STACKLEAK to the hardened config on 4.20+.

@joachifm @NeQuissimus

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Assured whether relevant documentation is up to date
  • Fits CONTRIBUTING.md.

@joachifm
Copy link
Contributor

joachifm commented Jan 4, 2019

I had no idea plugins had been disabled; I'd like to see them re-enabled unless there's a good reason not to.

@NeQuissimus
Copy link
Member

I disabled the above configs because there were build issues. It is possible they have been fixed in the meantime. Please do re-enable them, if all the kernels manage to build!

@NeQuissimus
Copy link
Member

NeQuissimus commented Jan 4, 2019

@GrahamcOfBorg build linux_4_4.configfile linux_4_9.configfile linux_4_14.configfile linux_4_19.configfile linux_4_20.configfile linux_testing.configfile linux_hardened.configfile linux_latest_hardened.configfile linux_latest-libre.configfile

@NeQuissimus
Copy link
Member

@GrahamcOfBorg test kernel-lts kernel-latest

@joachifm joachifm merged commit 893c51b into NixOS:master Jan 4, 2019
@joachifm
Copy link
Contributor

joachifm commented Jan 4, 2019

Thank you

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

Successfully merging this pull request may close these issues.

None yet

4 participants