Message ID | 500D0542.1020000@gmail.com |
---|---|
State | Accepted, archived |
Headers | show |
On Mon, Jul 23, 2012 at 04:03:14PM +0800, Wang Sheng-Hui wrote: > 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> Applied, thanks. (And apologies for the delay in getting back to you!) - Ted -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/fs/ext4/move_extent.c b/fs/ext4/move_extent.c index c5826c6..8383257 100644 --- a/fs/ext4/move_extent.c +++ b/fs/ext4/move_extent.c @@ -1002,7 +1002,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 "
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> --- fs/ext4/move_extent.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-)