diff options
| author | Christian Brauner <brauner@kernel.org> | 2025-03-05 11:08:11 +0100 |
|---|---|---|
| committer | Christian Brauner <brauner@kernel.org> | 2025-03-05 13:24:52 +0100 |
| commit | 816b2e602035214de9ca2f68751e64ecef023dd1 (patch) | |
| tree | 8978aaf2df5bce1124d6d7b9aab93241233c71d0 /fs/pidfs.c | |
| parent | b1e809e7f64ad47dd232ff072d8ef59c1fe414c5 (diff) | |
pidfs: switch to copy_struct_to_user()
We have a helper that deals with all the required logic.
Link: https://lore.kernel.org/r/20250305-work-pidfs-kill_on_last_close-v3-1-c8c3d8361705@kernel.org
Reviewed-by: Jeff Layton <jlayton@kernel.org>
Reviewed-by: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: Christian Brauner <brauner@kernel.org>
Diffstat (limited to 'fs/pidfs.c')
| -rw-r--r-- | fs/pidfs.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/fs/pidfs.c b/fs/pidfs.c index 049352f973de..aa8c8bda8c8f 100644 --- a/fs/pidfs.c +++ b/fs/pidfs.c @@ -276,10 +276,7 @@ static long pidfd_info(struct task_struct *task, unsigned int cmd, unsigned long * userspace knows about will be copied. If userspace provides a new * struct, only the bits that the kernel knows about will be copied. */ - if (copy_to_user(uinfo, &kinfo, min(usize, sizeof(kinfo)))) - return -EFAULT; - - return 0; + return copy_struct_to_user(uinfo, usize, &kinfo, sizeof(kinfo), NULL); } static bool pidfs_ioctl_valid(unsigned int cmd) |