AuxXxilium/linux_dsm_epyc7002
1
0
Fork 0
mirror of https://github.com/AuxXxilium/linux_dsm_epyc7002.git synced 2024-11-24 04:50:53 +07:00
Code Issues Actions Packages Projects Releases Wiki Activity

selinux: fix inode_doinit_with_dentry() LABEL_INVALID error handling

Browse Source 
[ Upstream commit 200ea5a2292dc444a818b096ae6a32ba3caa51b9 ]

A previous fix, commit 83370b31a915 ("selinux: fix error initialization
in inode_doinit_with_dentry()"), changed how failures were handled
before a SELinux policy was loaded.  Unfortunately that patch was
potentially problematic for two reasons: it set the isec->initialized
state without holding a lock, and it didn't set the inode's SELinux
label to the "default" for the particular filesystem.  The later can
be a problem if/when a later attempt to revalidate the inode fails
and SELinux reverts to the existing inode label.

This patch should restore the default inode labeling that existed
before the original fix, without affecting the LABEL_INVALID marking
such that revalidation will still be attempted in the future.

Fixes: 83370b31a915 ("selinux: fix error initialization in inode_doinit_with_dentry()")
Reported-by: Sven Schnelle <svens@linux.ibm.com>
Tested-by: Sven Schnelle <svens@linux.ibm.com>
Reviewed-by: Ondrej Mosnacek <omosnace@redhat.com>
Signed-off-by: Paul Moore <paul@paul-moore.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
Paul Moore 2020-11-03 11:49:38 -05:00 committed by Greg Kroah-Hartman
parent ae292e43b0
commit 6e5ea342fc
1 changed files with 13 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
security/selinux/hooks.c
View File

@ -1451,13 +1451,7 @@ static int inode_doinit_with_dentry(struct inode *inode, struct dentry *opt_dent
* inode_doinit with a dentry, before these inodes could * inode_doinit with a dentry, before these inodes could
* be used again by userspace. * be used again by userspace.
*/ */
isec->initialized = LABEL_INVALID; goto out_invalid;
/*
* There is nothing useful to jump to the "out"
* label, except a needless spin lock/unlock
* cycle.
*/
return 0;
} }
rc = inode_doinit_use_xattr(inode, dentry, sbsec->def_sid, rc = inode_doinit_use_xattr(inode, dentry, sbsec->def_sid,
@ -1513,15 +1507,8 @@ static int inode_doinit_with_dentry(struct inode *inode, struct dentry *opt_dent
* inode_doinit() with a dentry, before these inodes * inode_doinit() with a dentry, before these inodes
* could be used again by userspace. * could be used again by userspace.
*/ */
if (!dentry) { if (!dentry)
isec->initialized = LABEL_INVALID; goto out_invalid;
/*
* There is nothing useful to jump to the "out"
* label, except a needless spin lock/unlock
* cycle.
*/
return 0;
}
rc = selinux_genfs_get_sid(dentry, sclass, rc = selinux_genfs_get_sid(dentry, sclass,
sbsec->flags, &sid); sbsec->flags, &sid);
if (rc) { if (rc) {
@ -1546,11 +1533,10 @@ static int inode_doinit_with_dentry(struct inode *inode, struct dentry *opt_dent
out: out:
spin_lock(&isec->lock); spin_lock(&isec->lock);
if (isec->initialized == LABEL_PENDING) { if (isec->initialized == LABEL_PENDING) {
if (!sid || rc) { if (rc) {
isec->initialized = LABEL_INVALID; isec->initialized = LABEL_INVALID;
goto out_unlock; goto out_unlock;
} }
isec->initialized = LABEL_INITIALIZED; isec->initialized = LABEL_INITIALIZED;
isec->sid = sid; isec->sid = sid;
} }
@ -1558,6 +1544,15 @@ static int inode_doinit_with_dentry(struct inode *inode, struct dentry *opt_dent
out_unlock: out_unlock:
spin_unlock(&isec->lock); spin_unlock(&isec->lock);
return rc; return rc;
out_invalid:
spin_lock(&isec->lock);
if (isec->initialized == LABEL_PENDING) {
isec->initialized = LABEL_INVALID;
isec->sid = sid;
}
spin_unlock(&isec->lock);
return 0;
} }
/* Convert a Linux signal to an access vector. */ /* Convert a Linux signal to an access vector. */