diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2014-12-12 20:30:00 +0100 |
|---|---|---|
| committer | Chris Mason <clm@fb.com> | 2015-01-02 20:47:55 +0100 |
| commit | 6b6d24b38991f72fe974215c96e0fdfe409ea50e (patch) | |
| tree | 131d969fab0dc9c1f1452bb853134c411d5cd87b /fs/btrfs/scrub.c | |
| parent | Btrfs: add more maintainers (diff) | |
| download | linux-6b6d24b38991f72fe974215c96e0fdfe409ea50e.tar.xz linux-6b6d24b38991f72fe974215c96e0fdfe409ea50e.zip | |
Btrfs, scrub: uninitialized variable in scrub_extent_for_parity()
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>
Diffstat (limited to 'fs/btrfs/scrub.c')
| -rw-r--r-- | fs/btrfs/scrub.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/btrfs/scrub.c b/fs/btrfs/scrub.c index f2bb13a23f86..9e1569ffbf6e 100644 --- a/fs/btrfs/scrub.c +++ b/fs/btrfs/scrub.c @@ -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; |