diff options
| author | Amir Goldstein | 2019-06-05 17:04:49 +0200 |
|---|---|---|
| committer | Darrick J. Wong | 2019-06-09 19:06:19 +0200 |
| commit | 96e6e8f4a68df2d94800311163faa67124df24e5 (patch) | |
| tree | 42b63675c58ce53f9896daf674937625e71a684d /include/linux/fs.h | |
| parent | vfs: remove redundant checks from generic_remap_checks() (diff) | |
| download | kernel-qcow2-linux-96e6e8f4a68df2d94800311163faa67124df24e5.tar.gz kernel-qcow2-linux-96e6e8f4a68df2d94800311163faa67124df24e5.tar.xz kernel-qcow2-linux-96e6e8f4a68df2d94800311163faa67124df24e5.zip | |
vfs: add missing checks to copy_file_range
Like the clone and dedupe interfaces we've recently fixed, the
copy_file_range() implementation is missing basic sanity, limits and
boundary condition tests on the parameters that are passed to it
from userspace. Create a new "generic_copy_file_checks()" function
modelled on the generic_remap_checks() function to provide this
missing functionality.
[Amir] Shorten copy length instead of checking pos_in limits
because input file size already abides by the limits.
Signed-off-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Amir Goldstein <amir73il@gmail.com>
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Diffstat (limited to 'include/linux/fs.h')
| -rw-r--r-- | include/linux/fs.h | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/include/linux/fs.h b/include/linux/fs.h index 89b9b73eb581..e4d382c4342a 100644 --- a/include/linux/fs.h +++ b/include/linux/fs.h @@ -3050,6 +3050,9 @@ extern int generic_remap_checks(struct file *file_in, loff_t pos_in, struct file *file_out, loff_t pos_out, loff_t *count, unsigned int remap_flags); extern int generic_file_rw_checks(struct file *file_in, struct file *file_out); +extern int generic_copy_file_checks(struct file *file_in, loff_t pos_in, + struct file *file_out, loff_t pos_out, + size_t *count, unsigned int flags); extern ssize_t generic_file_read_iter(struct kiocb *, struct iov_iter *); extern ssize_t __generic_file_write_iter(struct kiocb *, struct iov_iter *); extern ssize_t generic_file_write_iter(struct kiocb *, struct iov_iter *); |