diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2022-03-15 13:30:09 +0300 |
---|---|---|
committer | Chuck Lever <chuck.lever@oracle.com> | 2022-03-15 09:35:56 -0400 |
commit | 184416d4b98509fb4c3d8fc3d6dc1437896cc159 (patch) | |
tree | 142754edc3ddc62e00f7f297df7ab6028853e94f /fs/nfsd/nfsproc.c | |
parent | 9d6647762b9c6b555bc83d97d7c93be6057a990f (diff) | |
download | lwn-184416d4b98509fb4c3d8fc3d6dc1437896cc159.tar.gz lwn-184416d4b98509fb4c3d8fc3d6dc1437896cc159.zip |
NFSD: prevent underflow in nfssvc_decode_writeargs()
Smatch complains:
fs/nfsd/nfsxdr.c:341 nfssvc_decode_writeargs()
warn: no lower bound on 'args->len'
Change the type to unsigned to prevent this issue.
Cc: stable@vger.kernel.org
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Diffstat (limited to 'fs/nfsd/nfsproc.c')
-rw-r--r-- | fs/nfsd/nfsproc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/nfsd/nfsproc.c b/fs/nfsd/nfsproc.c index 18b8eb43a19b..fcdab8a8a41f 100644 --- a/fs/nfsd/nfsproc.c +++ b/fs/nfsd/nfsproc.c @@ -230,7 +230,7 @@ nfsd_proc_write(struct svc_rqst *rqstp) unsigned long cnt = argp->len; unsigned int nvecs; - dprintk("nfsd: WRITE %s %d bytes at %d\n", + dprintk("nfsd: WRITE %s %u bytes at %d\n", SVCFH_fmt(&argp->fh), argp->len, argp->offset); |