diff options
| author |   Amir Goldstein <amir73il@gmail.com> | 2019-06-05 08:04:48 -0700 |
|---|---|---|
| committer |   Darrick J. Wong <darrick.wong@oracle.com> | 2019-06-09 10:06:19 -0700 |
| commit | 646955cd5425dd8fed8205cbb1b4373c222d028e (patch) | |
| tree | b05036cc5e7433a38305db3f6910bc7905bc067a /fs/read_write.c | |
| parent | vfs: introduce generic_file_rw_checks() (diff) | |
| download | linux-dev-646955cd5425dd8fed8205cbb1b4373c222d028e.tar.xz linux-dev-646955cd5425dd8fed8205cbb1b4373c222d028e.zip | |
vfs: remove redundant checks from generic_remap_checks()
The access limit checks on input file range in generic_remap_checks()
are redundant because the input file size is guaranteed to be within
limits and pos+len are already checked to be within input file size.
Beyond the fact that the check cannot fail, if it would have failed,
it could return -EFBIG for input file range error. There is no precedent
for that. -EFBIG is returned in syscalls that would change file length.
With that call removed, we can fold generic_access_check_limits() into
generic_write_check_limits().
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 'fs/read_write.c')
0 files changed, 0 insertions, 0 deletions