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 /net/caif/caif_socket.c | |
| parent | a31713517dac0862a3f0ec9006df9160ce022b0c (diff) | |
| download | lwn-646955cd5425dd8fed8205cbb1b4373c222d028e.tar.gz lwn-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 'net/caif/caif_socket.c')
0 files changed, 0 insertions, 0 deletions