Re: [PATCH] security: selinux: allow per-file labeling for cgroupfs

[Stephen Smalley]

On Fri, 2017-03-10 at 15:01 -0500, Paul Moore wrote:
> On Thu, Feb 9, 2017 at 10:58 AM, Antonio Murdaca <amurdaca@xxxxxxxxxx
> > wrote:
> > 
> > This patch allows genfscon per-file labeling for cgroupfs. For
> > instance,
> > this allows to label the "release_agent" file within each
> > cgroup mount and limit writes to it.
> > 
> > Signed-off-by: Antonio Murdaca <runcom@xxxxxxxxxx>
> > ---
> > Âsecurity/selinux/hooks.c | 2 ++
> > Â1 file changed, 2 insertions(+)
> 
> Now that the merge window is behind us, let's get this merged, but
> could you update it to use the selinux_policycap_cgroupseclabel
> policy
> capability?ÂÂSee 2651225b5ebcdde ("selinux: wrap cgroup seclabel
> support with its own policy capability") for more information.

I don't think that is necessary. ÂThis change unlike the other one
should not yield any difference in behavior with existing policy; it
just allows one to specify fine-grained labeling for cgroup nodes in
future policy. ÂIt doesn't affect any userspace interface.


> Also, how goes the testing?
> 
> > 
> > diff --git a/security/selinux/hooks.c b/security/selinux/hooks.c
> > index 9a8f12f..5a3138e 100644
> > --- a/security/selinux/hooks.c
> > +++ b/security/selinux/hooks.c
> > @@ -808,6 +808,8 @@ static int selinux_set_mnt_opts(struct
> > super_block *sb,
> > 
> > ÂÂÂÂÂÂÂÂif (!strcmp(sb->s_type->name, "debugfs") ||
> > ÂÂÂÂÂÂÂÂÂÂÂÂ!strcmp(sb->s_type->name, "sysfs") ||
> > +ÂÂÂÂÂÂÂÂÂÂÂ!strcmp(sb->s_type->name, "cgroup") ||
> > +ÂÂÂÂÂÂÂÂÂÂÂ!strcmp(sb->s_type->name, "cgroup2") ||
> > ÂÂÂÂÂÂÂÂÂÂÂÂ!strcmp(sb->s_type->name, "pstore"))
> > ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂsbsec->flags |= SE_SBGENFS;
> > 
> > --
> > 2.9.3
> > 
> > _______________________________________________
> > Selinux mailing list
> > Selinux@xxxxxxxxxxxxx
> > To unsubscribe, send email to Selinux-leave@xxxxxxxxxxxxxx
> > To get help, send an email containing "help" to Selinux-request@tyc
> > ho.nsa.gov.
>