Commit e06b6405 authored by Chuck Lever's avatar Chuck Lever Committed by J. Bruce Fields

NFSD: Stricter buffer size checking in fs/nfsd/nfsctl.c

Clean up: For consistency, handle output buffer size checking in a
other nfsctl functions the same way it's done for write_versions().
Signed-off-by: default avatarChuck Lever <chuck.lever@oracle.com>
Signed-off-by: default avatarJ. Bruce Fields <bfields@citi.umich.edu>
parent 261758b5
...@@ -695,8 +695,9 @@ static ssize_t write_threads(struct file *file, char *buf, size_t size) ...@@ -695,8 +695,9 @@ static ssize_t write_threads(struct file *file, char *buf, size_t size)
if (rv) if (rv)
return rv; return rv;
} }
sprintf(buf, "%d\n", nfsd_nrthreads());
return strlen(buf); return scnprintf(buf, SIMPLE_TRANSACTION_LIMIT, "%d\n",
nfsd_nrthreads());
} }
/** /**
...@@ -1197,7 +1198,9 @@ static ssize_t write_maxblksize(struct file *file, char *buf, size_t size) ...@@ -1197,7 +1198,9 @@ static ssize_t write_maxblksize(struct file *file, char *buf, size_t size)
nfsd_max_blksize = bsize; nfsd_max_blksize = bsize;
mutex_unlock(&nfsd_mutex); mutex_unlock(&nfsd_mutex);
} }
return sprintf(buf, "%d\n", nfsd_max_blksize);
return scnprintf(buf, SIMPLE_TRANSACTION_LIMIT, "%d\n",
nfsd_max_blksize);
} }
#ifdef CONFIG_NFSD_V4 #ifdef CONFIG_NFSD_V4
...@@ -1221,8 +1224,9 @@ static ssize_t __write_leasetime(struct file *file, char *buf, size_t size) ...@@ -1221,8 +1224,9 @@ static ssize_t __write_leasetime(struct file *file, char *buf, size_t size)
return -EINVAL; return -EINVAL;
nfs4_reset_lease(lease); nfs4_reset_lease(lease);
} }
sprintf(buf, "%ld\n", nfs4_lease_time());
return strlen(buf); return scnprintf(buf, SIMPLE_TRANSACTION_LIMIT, "%ld\n",
nfs4_lease_time());
} }
/** /**
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment