diff options
author | Fabian Frederick <fabf@skynet.be> | 2017-04-25 14:11:02 -0400 |
---|---|---|
committer | David Sterba <dsterba@suse.com> | 2017-06-19 12:25:57 -0400 |
commit | 977ec79271fb6dfa0f853ea5aa3c80d3202893ee (patch) | |
tree | 742ec4c2b856697b91fa9de7cedfc7edf1475816 /fs/btrfs/check-integrity.c | |
parent | 41f1830f5a7af77cf5c86359aba3cbd706687e52 (diff) |
btrfs: kmap() can't fail
Remove NULL test on kmap() as it will always return a valid pointer.
Signed-off-by: Fabian Frederick <fabf@skynet.be>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/check-integrity.c')
-rw-r--r-- | fs/btrfs/check-integrity.c | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/fs/btrfs/check-integrity.c b/fs/btrfs/check-integrity.c index ab14c2e635ca..496eb009c41d 100644 --- a/fs/btrfs/check-integrity.c +++ b/fs/btrfs/check-integrity.c | |||
@@ -1668,14 +1668,8 @@ static int btrfsic_read_block(struct btrfsic_state *state, | |||
1668 | dev_bytenr += (j - i) * PAGE_SIZE; | 1668 | dev_bytenr += (j - i) * PAGE_SIZE; |
1669 | i = j; | 1669 | i = j; |
1670 | } | 1670 | } |
1671 | for (i = 0; i < num_pages; i++) { | 1671 | for (i = 0; i < num_pages; i++) |
1672 | block_ctx->datav[i] = kmap(block_ctx->pagev[i]); | 1672 | block_ctx->datav[i] = kmap(block_ctx->pagev[i]); |
1673 | if (!block_ctx->datav[i]) { | ||
1674 | pr_info("btrfsic: kmap() failed (dev %s)!\n", | ||
1675 | block_ctx->dev->name); | ||
1676 | return -1; | ||
1677 | } | ||
1678 | } | ||
1679 | 1673 | ||
1680 | return block_ctx->len; | 1674 | return block_ctx->len; |
1681 | } | 1675 | } |