diff options
author | Zhao Lei <zhaolei@cn.fujitsu.com> | 2015-01-20 02:11:35 -0500 |
---|---|---|
committer | Chris Mason <clm@fb.com> | 2015-01-21 21:06:48 -0500 |
commit | b25c94c580c27bb6cff1e2f478746e77119215ad (patch) | |
tree | c2e72e3bb442e60bf2f604efeecaf8b34e27afe6 /fs/btrfs/scrub.c | |
parent | 6e9606d2a2dce098c1739fb3cd82a1c34fd73d3a (diff) |
Btrfs: Fix a jump typo of nodatasum_case to avoid wrong WARN_ON()
if (sctx->is_dev_replace && !is_metadata && !have_csum) {
...
goto nodatasum_case;
}
...
nodatasum_case:
WARN_ON(sctx->is_dev_replace);
In above code, nodatasum_case marker should be moved after
WARN_ON().
Signed-off-by: Zhao Lei <zhaolei@cn.fujitsu.com>
Signed-off-by: Miao Xie <miaox@cn.fujitsu.com>
Signed-off-by: Chris Mason <clm@fb.com>
Diffstat (limited to 'fs/btrfs/scrub.c')
-rw-r--r-- | fs/btrfs/scrub.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/btrfs/scrub.c b/fs/btrfs/scrub.c index 1388e7127eea..016512c590ea 100644 --- a/fs/btrfs/scrub.c +++ b/fs/btrfs/scrub.c | |||
@@ -1036,9 +1036,10 @@ static int scrub_handle_errored_block(struct scrub_block *sblock_to_check) | |||
1036 | if (!is_metadata && !have_csum) { | 1036 | if (!is_metadata && !have_csum) { |
1037 | struct scrub_fixup_nodatasum *fixup_nodatasum; | 1037 | struct scrub_fixup_nodatasum *fixup_nodatasum; |
1038 | 1038 | ||
1039 | nodatasum_case: | ||
1040 | WARN_ON(sctx->is_dev_replace); | 1039 | WARN_ON(sctx->is_dev_replace); |
1041 | 1040 | ||
1041 | nodatasum_case: | ||
1042 | |||
1042 | /* | 1043 | /* |
1043 | * !is_metadata and !have_csum, this means that the data | 1044 | * !is_metadata and !have_csum, this means that the data |
1044 | * might not be COW'ed, that it might be modified | 1045 | * might not be COW'ed, that it might be modified |