ocfs2: checking the wrong variable in ocfs2_move_extent()
"new_phys_cpos" is always a valid pointer here. ocfs2_probe_alloc_group() allocates "*new_phys_cpos". Signed-off-by: Dan Carpenter <error27@gmail.com> Signed-off-by: Joel Becker <jlbec@evilplan.org>
This commit is contained in:
committed by
Joel Becker
parent
03efed8a2a
commit
3d75be7c47
@@ -746,7 +746,7 @@ static int ocfs2_move_extent(struct ocfs2_move_extents_context *context,
|
|||||||
*/
|
*/
|
||||||
ocfs2_probe_alloc_group(inode, gd_bh, &goal_bit, len, move_max_hop,
|
ocfs2_probe_alloc_group(inode, gd_bh, &goal_bit, len, move_max_hop,
|
||||||
new_phys_cpos);
|
new_phys_cpos);
|
||||||
if (!new_phys_cpos) {
|
if (!*new_phys_cpos) {
|
||||||
ret = -ENOSPC;
|
ret = -ENOSPC;
|
||||||
goto out_commit;
|
goto out_commit;
|
||||||
}
|
}
|
||||||
|
Reference in New Issue
Block a user