Re: [PATCH 4/5] ksmbd: don't open-code %pf

From: Namjae Jeon
Date: Thu Aug 18 2022 - 02:08:44 EST


2022-08-18 11:59 GMT+09:00, Al Viro <viro@xxxxxxxxxxxxxxxxxx>:
> Signed-off-by: Al Viro <viro@xxxxxxxxxxxxxxxxxx>
> ---
> fs/ksmbd/vfs.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/fs/ksmbd/vfs.c b/fs/ksmbd/vfs.c
> index 78d01033604c..a0fafba8b5d0 100644
> --- a/fs/ksmbd/vfs.c
> +++ b/fs/ksmbd/vfs.c
> @@ -1743,11 +1743,11 @@ int ksmbd_vfs_copy_file_ranges(struct ksmbd_work
> *work,
> *total_size_written = 0;
>
> if (!(src_fp->daccess & (FILE_READ_DATA_LE | FILE_EXECUTE_LE))) {
> - pr_err("no right to read(%pd)\n", src_fp->filp->f_path.dentry);
> + pr_err("no right to read(%pf)\n", src_fp->filp);
Isn't this probably %pD?

Thanks.
> return -EACCES;
> }
> if (!(dst_fp->daccess & (FILE_WRITE_DATA_LE | FILE_APPEND_DATA_LE))) {
> - pr_err("no right to write(%pd)\n", dst_fp->filp->f_path.dentry);
> + pr_err("no right to write(%pf)\n", dst_fp->filp);
> return -EACCES;
> }
>
> --
> 2.30.2
>
>