diff options
| author | Wang Sheng-Hui <shhuiw@gmail.com> | 2012-09-27 08:00:01 -0400 |
|---|---|---|
| committer | Theodore Ts'o <tytso@mit.edu> | 2012-09-27 08:00:01 -0400 |
| commit | cbb4ee830e0057f8d60b4e0a8c4b6daa6cdd28d7 (patch) | |
| tree | 08a92472e8ee9131a3ce462b2545872625a75cca /fs/ext4 | |
| parent | c25f9bc6143f4cb4dc31d2ad7a6fe4e4005fc414 (diff) | |
| download | linux-next-cbb4ee830e0057f8d60b4e0a8c4b6daa6cdd28d7.tar.gz | |
ext4: remove redundant offset check in mext_check_arguments()
In the check code above, if orig_start != donor_start, we would
return -EINVAL. So here, orig_start should be equal with donor_start.
Remove the redundant check here.
Signed-off-by: Wang Sheng-Hui <shhuiw@gmail.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'fs/ext4')
| -rw-r--r-- | fs/ext4/move_extent.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/ext4/move_extent.c b/fs/ext4/move_extent.c index c2e47da7c2ba..cee4bd066b7a 100644 --- a/fs/ext4/move_extent.c +++ b/fs/ext4/move_extent.c @@ -1134,7 +1134,6 @@ mext_check_arguments(struct inode *orig_inode, } if ((orig_start >= EXT_MAX_BLOCKS) || - (donor_start >= EXT_MAX_BLOCKS) || (*len > EXT_MAX_BLOCKS) || (orig_start + *len >= EXT_MAX_BLOCKS)) { ext4_debug("ext4 move extent: Can't handle over [%u] blocks " |