Unverified Commit 07162dde authored by openeuler-ci-bot's avatar openeuler-ci-bot Committed by Gitee
Browse files

!5921 ext4: fix double-free of blocks due to wrong extents moved_len 

Merge Pull Request from: @ci-robot 
 
PR sync from: Long Li <leo.lilong@huawei.com>
https://mailweb.openeuler.org/hyperkitty/list/kernel@openeuler.org/message/2ECRXQO35D5U7ONWLRQQ4GXAONG2GKRV/ 
 
https://gitee.com/src-openeuler/kernel/issues/I9E2F9 
 
Link:https://gitee.com/openeuler/kernel/pulls/5921

 

Reviewed-by: default avatarJialin Zhang <zhangjialin11@huawei.com>
Signed-off-by: default avatarJialin Zhang <zhangjialin11@huawei.com>
parents c0b982ec f5999e84

fs/ext4/move_extent.c

+2 −4
Original line number Diff line number Diff line
@@ -615,6 +615,7 @@ ext4_move_extents(struct file *o_filp, struct file *d_filp, __u64 orig_blk, 
		goto out;

	o_end = o_start + len;



	*moved_len = 0;

	while (o_start < o_end) {

		struct ext4_extent *ex;

		ext4_lblk_t cur_blk, next_blk;
@@ -670,7 +671,7 @@ ext4_move_extents(struct file *o_filp, struct file *d_filp, __u64 orig_blk, 
		 */

		ext4_double_up_write_data_sem(orig_inode, donor_inode);

		/* Swap original branches with new branches */

		move_extent_per_page(o_filp, donor_inode,

		*moved_len += move_extent_per_page(o_filp, donor_inode,

				     orig_page_index, donor_page_index,

				     offset_in_page, cur_len,

				     unwritten, &ret);
@@ -680,9 +681,6 @@ ext4_move_extents(struct file *o_filp, struct file *d_filp, __u64 orig_blk, 
		o_start += cur_len;

		d_start += cur_len;

	}

	*moved_len = o_start - orig_blk;

	if (*moved_len > len)

		*moved_len = len;



out:

	if (*moved_len) {