Re: [PATCH v3 1/3] nfsd: ignore requests to disable unsupported versions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Stylistically, I think this needs a "break;", but otherwise

Reviewed-by: Tom Talpey <tom@xxxxxxxxxx>

On 10/18/2022 7:47 AM, Jeff Layton wrote:
The kernel currently errors out if you attempt to enable or disable a
version that it doesn't recognize. Change it to ignore attempts to
disable an unrecognized version. If we don't support it, then there is
no harm in doing so.

Signed-off-by: Jeff Layton <jlayton@xxxxxxxxxx>
---
  fs/nfsd/nfsctl.c | 4 +++-
  1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/fs/nfsd/nfsctl.c b/fs/nfsd/nfsctl.c
index dc74a947a440..68ed42fd29fc 100644
--- a/fs/nfsd/nfsctl.c
+++ b/fs/nfsd/nfsctl.c
@@ -601,7 +601,9 @@ static ssize_t __write_versions(struct file *file, char *buf, size_t size)
  				}
  				break;
  			default:
-				return -EINVAL;
+				/* Ignore requests to disable non-existent versions */
+				if (cmd == NFSD_SET)
+					return -EINVAL;
  			}
  			vers += len + 1;
  		} while ((len = qword_get(&mesg, vers, size)) > 0);



[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux