Skip to content

Commit

Permalink
Btrfs, scrub: uninitialized variable in scrub_extent_for_parity()
Browse files Browse the repository at this point in the history
The only way that "ret" is set is when we call scrub_pages_for_parity()
so the skip to "if (ret) " test doesn't make sense and causes a static
checker warning.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Chris Mason <clm@fb.com>
  • Loading branch information
Dan Carpenter authored and Chris Mason committed Jan 2, 2015
1 parent 4e0c4a4 commit 6b6d24b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion fs/btrfs/scrub.c
Original file line number Diff line number Diff line change
Expand Up @@ -2607,9 +2607,9 @@ static int scrub_extent_for_parity(struct scrub_parity *sparity,
ret = scrub_pages_for_parity(sparity, logical, l, physical, dev,
flags, gen, mirror_num,
have_csum ? csum : NULL);
skip:
if (ret)
return ret;
skip:
len -= l;
logical += l;
physical += l;
Expand Down

0 comments on commit 6b6d24b

Please sign in to comment.