| Message ID | 464519.1579276102@warthog.procyon.org.uk (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | nfs: Return EINVAL rather than ERANGE for mount parse errors | expand |
diff --git a/fs/nfs/fs_context.c b/fs/nfs/fs_context.c index 429315c011ae..07cbd655dafb 100644 --- a/fs/nfs/fs_context.c +++ b/fs/nfs/fs_context.c @@ -770,7 +770,7 @@ static int nfs_fs_context_parse_param(struct fs_context *fc, return nfs_invalf(fc, "NFS: Bad IP address specified"); out_of_bounds: nfs_invalf(fc, "NFS: Value for '%s' out of range", param->key); - return -ERANGE; + return -EINVAL; } /*
Hi Krzysztof, Does this patch fix the problem? David --- commit 3021f58ee1e2c9659e629d0ccf06d3e0876e805a Author: David Howells <dhowells@redhat.com> Date: Fri Jan 17 15:37:46 2020 +0000 nfs: Return EINVAL rather than ERANGE for mount parse errors Return EINVAL rather than ERANGE for mount parse errors as the userspace mount command doesn't necessarily understand what to do with anything other than EINVAL. The old code returned -ERANGE as an intermediate error that then get converted to -EINVAL, whereas the new code returns -ERANGE. This was induced by passing minorversion=1 to a v4 mount where CONFIG_NFS_V4_1 was disabled in the kernel build. Fixes: 68f65ef40e1e ("NFS: Convert mount option parsing to use functionality from fs_parser.h") Reported-by: Krzysztof Kozlowski <krzk@kernel.org> Signed-off-by: David Howells <dhowells@redhat.com>