2019-05-27 12:22:41

by Geert Uytterhoeven

[permalink] [raw]
Subject: [PATCH] [trivial] nfsd: Spelling s/EACCESS/EACCES/

The correct spelling is EACCES:

include/uapi/asm-generic/errno-base.h:#define EACCES 13 /* Permission denied */

Signed-off-by: Geert Uytterhoeven <[email protected]>
---
fs/nfsd/vfs.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/nfsd/vfs.c b/fs/nfsd/vfs.c
index fc24ee47eab51ad4..c85783e536d595de 100644
--- a/fs/nfsd/vfs.c
+++ b/fs/nfsd/vfs.c
@@ -404,7 +404,7 @@ nfsd_setattr(struct svc_rqst *rqstp, struct svc_fh *fhp, struct iattr *iap,
/*
* If utimes(2) and friends are called with times not NULL, we should
* not set NFSD_MAY_WRITE bit. Otherwise fh_verify->nfsd_permission
- * will return EACCESS, when the caller's effective UID does not match
+ * will return EACCES, when the caller's effective UID does not match
* the owner of the file, and the caller is not privileged. In this
* situation, we should return EPERM(notify_change will return this).
*/
--
2.17.1


2019-05-31 13:37:08

by J. Bruce Fields

[permalink] [raw]
Subject: Re: [PATCH] [trivial] nfsd: Spelling s/EACCESS/EACCES/

Thanks, applying for 5.3.--b.

On Mon, May 27, 2019 at 02:21:32PM +0200, Geert Uytterhoeven wrote:
> The correct spelling is EACCES:
>
> include/uapi/asm-generic/errno-base.h:#define EACCES 13 /* Permission denied */
>
> Signed-off-by: Geert Uytterhoeven <[email protected]>
> ---
> fs/nfsd/vfs.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/nfsd/vfs.c b/fs/nfsd/vfs.c
> index fc24ee47eab51ad4..c85783e536d595de 100644
> --- a/fs/nfsd/vfs.c
> +++ b/fs/nfsd/vfs.c
> @@ -404,7 +404,7 @@ nfsd_setattr(struct svc_rqst *rqstp, struct svc_fh *fhp, struct iattr *iap,
> /*
> * If utimes(2) and friends are called with times not NULL, we should
> * not set NFSD_MAY_WRITE bit. Otherwise fh_verify->nfsd_permission
> - * will return EACCESS, when the caller's effective UID does not match
> + * will return EACCES, when the caller's effective UID does not match
> * the owner of the file, and the caller is not privileged. In this
> * situation, we should return EPERM(notify_change will return this).
> */
> --
> 2.17.1