Commit 7bb185ed authored by Stephen Smalley's avatar Stephen Smalley Committed by Paul Moore

selinux: fix mounting of cgroup2 under older policies

commit 901ef845 ("selinux: allow per-file labeling for cgroupfs")
broke mounting of cgroup2 under older SELinux policies which lacked
a genfscon rule for cgroup2.  This prevents mounting of cgroup2 even
when SELinux is permissive.

Change the handling when there is no genfscon rule in policy to
just mark the inode unlabeled and not return an error to the caller.
This permits mounting and access if allowed by policy, e.g. to
unconfined domains.

I also considered changing the behavior of security_genfs_sid() to
never return -ENOENT, but the current behavior is relied upon by
other callers to perform caller-specific handling.

Fixes: 901ef845 ("selinux: allow per-file labeling for cgroupfs")
CC: <stable@vger.kernel.org>
Reported-by: default avatarDmitry Vyukov <dvyukov@google.com>
Reported-by: default avatarWaiman Long <longman@redhat.com>
Signed-off-by: default avatarStephen Smalley <sds@tycho.nsa.gov>
Tested-by: default avatarWaiman Long <longman@redhat.com>
Signed-off-by: default avatarPaul Moore <paul@paul-moore.com>
parent 57361846
......@@ -1508,6 +1508,11 @@ static int selinux_genfs_get_sid(struct dentry *dentry,
}
rc = security_genfs_sid(&selinux_state, sb->s_type->name,
path, tclass, sid);
if (rc == -ENOENT) {
/* No match in policy, mark as unlabeled. */
*sid = SECINITSID_UNLABELED;
rc = 0;
}
}
free_page((unsigned long)buffer);
return rc;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment