aboutsummaryrefslogtreecommitdiffstats
path: root/fs/ocfs2
diff options
context:
space:
mode:
authorDan Carpenter <error27@gmail.com>2011-05-29 15:56:31 -0400
committerJoel Becker <jlbec@evilplan.org>2011-05-31 22:03:44 -0400
commit3d75be7c4771c7e4d5b5fa586a599af8473de32c (patch)
tree45052c6e7317f1bfe2d0f6961ca5a738deae5766 /fs/ocfs2
parent03efed8a2a1b8e00164eb4720a82a7dd5e368a8e (diff)
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>
Diffstat (limited to 'fs/ocfs2')
-rw-r--r--fs/ocfs2/move_extents.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/ocfs2/move_extents.c b/fs/ocfs2/move_extents.c
index cd9427023d2e..d3433d60dbb9 100644
--- a/fs/ocfs2/move_extents.c
+++ b/fs/ocfs2/move_extents.c
@@ -746,7 +746,7 @@ static int ocfs2_move_extent(struct ocfs2_move_extents_context *context,
746 */ 746 */
747 ocfs2_probe_alloc_group(inode, gd_bh, &goal_bit, len, move_max_hop, 747 ocfs2_probe_alloc_group(inode, gd_bh, &goal_bit, len, move_max_hop,
748 new_phys_cpos); 748 new_phys_cpos);
749 if (!new_phys_cpos) { 749 if (!*new_phys_cpos) {
750 ret = -ENOSPC; 750 ret = -ENOSPC;
751 goto out_commit; 751 goto out_commit;
752 } 752 }