Skip to content

Commit bd14e46

Browse files
alberandbrauner
authored andcommitted
selinux: implement inode_file_[g|s]etattr hooks
These hooks are called on inode extended attribute retrieval/change. Cc: selinux@vger.kernel.org Cc: Paul Moore <paul@paul-moore.com> Acked-by: Paul Moore <paul@paul-moore.com> Signed-off-by: Andrey Albershteyn <aalbersh@kernel.org> Link: https://lore.kernel.org/20250630-xattrat-syscall-v6-3-c4e3bc35227b@kernel.org Signed-off-by: Christian Brauner <brauner@kernel.org>
1 parent defdd02 commit bd14e46

1 file changed

Lines changed: 14 additions & 0 deletions

File tree

security/selinux/hooks.c

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3480,6 +3480,18 @@ static int selinux_inode_removexattr(struct mnt_idmap *idmap,
34803480
return -EACCES;
34813481
}
34823482

3483+
static int selinux_inode_file_setattr(struct dentry *dentry,
3484+
struct fileattr *fa)
3485+
{
3486+
return dentry_has_perm(current_cred(), dentry, FILE__SETATTR);
3487+
}
3488+
3489+
static int selinux_inode_file_getattr(struct dentry *dentry,
3490+
struct fileattr *fa)
3491+
{
3492+
return dentry_has_perm(current_cred(), dentry, FILE__GETATTR);
3493+
}
3494+
34833495
static int selinux_path_notify(const struct path *path, u64 mask,
34843496
unsigned int obj_type)
34853497
{
@@ -7350,6 +7362,8 @@ static struct security_hook_list selinux_hooks[] __ro_after_init = {
73507362
LSM_HOOK_INIT(inode_getxattr, selinux_inode_getxattr),
73517363
LSM_HOOK_INIT(inode_listxattr, selinux_inode_listxattr),
73527364
LSM_HOOK_INIT(inode_removexattr, selinux_inode_removexattr),
7365+
LSM_HOOK_INIT(inode_file_getattr, selinux_inode_file_getattr),
7366+
LSM_HOOK_INIT(inode_file_setattr, selinux_inode_file_setattr),
73537367
LSM_HOOK_INIT(inode_set_acl, selinux_inode_set_acl),
73547368
LSM_HOOK_INIT(inode_get_acl, selinux_inode_get_acl),
73557369
LSM_HOOK_INIT(inode_remove_acl, selinux_inode_remove_acl),

0 commit comments

Comments
 (0)