Thread (92 messages) 92 messages, 7 authors, 2018-10-08

Re: [PATCH security-next v4 23/32] selinux: Remove boot parameter

From: James Morris <jmorris@namei.org>
Date: 2018-10-05 04:58:45
Also in: linux-arch, linux-doc, lkml

On Thu, 4 Oct 2018, Kees Cook wrote:
On Thu, Oct 4, 2018 at 10:49 AM, James Morris [off-list ref] wrote:
quoted
On Wed, 3 Oct 2018, Kees Cook wrote:
quoted
Then someone boots the system with:

selinux=1 security=selinux

In what order does selinux get initialized relative to yama?
(apparmor, flagged as a "legacy major", would have been disabled by
the "security=" not matching it.)
It doesn't, it needs to be specified in one place.

Distros will need to update boot parameter handling for this kernel
onwards.  Otherwise, we will need to carry this confusing mess forward
forever.
Are you saying that you want to overrule Paul and Stephen about
keeping "selinux=1 secuiryt=selinux" working?
Not overrule, but convince.

At least, deprecate selinux=1 and security=X, but not extend it any 
further.
quoted
In my most recent suggestion, there is no '!' disablement, just
enablement.  If an LSM is not listed in CONFIG_LSM="", it's not enabled.
And a user would need to specify ALL lsms on the "lsm=" line?
Yes, the ones they want enabled.
What do you think of my latest proposal? It could happily work all
three ways: old boot params and security= work ("selinux=1
security=selinux" keeps working), individual LSM enable/disable works
("lsm=+loadpin"), and full LSM ordering works
("lsm=each,lsm,in,order,here"):

https://lore.kernel.org/lkml/CAGXu5jJJit8bDNvgXaFkuvFPy7NWtJW2oRWFbG-6iWk0+A1qng@mail.gmail.com/ (local)
I think having something like +yama will still lead to confusion.
Explicitly stating each enabled LSM in order is totally unambiguous.

If people are moving away from the distro defaults, and there is no 
high-level interface to manage this, it seems to me there's a deeper 
issue with the distro.


-- 
James Morris
[off-list ref]
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help