Re: [PATCH] fs,security: Fix file_set_fowner LSM hook inconsistencies
From: Paul Moore <paul@paul-moore.com>
Date: 2024-08-12 15:00:40
Also in:
linux-fsdevel, lkml, selinux
On Mon, Aug 12, 2024 at 10:49 AM Mickaël Salaün [off-list ref] wrote:
quoted hunk ↗ jump to hunk
diff --git a/include/linux/lsm_hook_defs.h b/include/linux/lsm_hook_defs.h index 44488b1ab9a9..974bcc1c8f8f 100644 --- a/include/linux/lsm_hook_defs.h +++ b/include/linux/lsm_hook_defs.h@@ -196,7 +196,6 @@ LSM_HOOK(int, 0, file_mprotect, struct vm_area_struct *vma, LSM_HOOK(int, 0, file_lock, struct file *file, unsigned int cmd) LSM_HOOK(int, 0, file_fcntl, struct file *file, unsigned int cmd, unsigned long arg) -LSM_HOOK(void, LSM_RET_VOID, file_set_fowner, struct file *file)
As I mentioned in the other thread, I don't want to see the file_set_owner hook removed at this point in time. I'm open to the idea of moving it around, but as of right now I think it is important to keep it around.
LSM_HOOK(int, 0, file_send_sigiotask, struct task_struct *tsk,
struct fown_struct *fown, int sig)
LSM_HOOK(int, 0, file_receive, struct file *file)-- paul-moore.com