On Tue, 2025-03-04 at 10:41 +0100, Christian Brauner wrote: > We have a helper that deals with all the required logic. > > Signed-off-by: Christian Brauner <brauner@xxxxxxxxxx> > --- > fs/pidfs.c | 5 +---- > 1 file changed, 1 insertion(+), 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) > Reviewed-by: Jeff Layton <jlayton@xxxxxxxxxx>
