diff options
| author | Tsutomu Itoh <t-itoh@jp.fujitsu.com> | 2012-09-06 13:08:59 +0400 |
|---|---|---|
| committer | Chris Mason <chris.mason@fusionio.com> | 2012-10-01 23:19:14 +0400 |
| commit | f54fb859da53f04a443c5e3f4cb9b936ed42d227 (patch) | |
| tree | 21fba36b78cf33ada070fdfea0529cac8b4b6194 /fs/btrfs/relocation.c | |
| parent | 903889f462409c816893abd02d88636f7b4a7774 (diff) | |
| download | linux-f54fb859da53f04a443c5e3f4cb9b936ed42d227.tar.xz | |
Btrfs: fix error handling in delete_block_group_cache()
btrfs_iget() never return NULL.
So, NULL check is unnecessary.
Signed-off-by: Tsutomu Itoh <t-itoh@jp.fujitsu.com>
Diffstat (limited to 'fs/btrfs/relocation.c')
| -rw-r--r-- | fs/btrfs/relocation.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/btrfs/relocation.c b/fs/btrfs/relocation.c index 5a15c96a18ff..7e7fd1bcfc54 100644 --- a/fs/btrfs/relocation.c +++ b/fs/btrfs/relocation.c @@ -3270,8 +3270,8 @@ static int delete_block_group_cache(struct btrfs_fs_info *fs_info, key.offset = 0; inode = btrfs_iget(fs_info->sb, &key, root, NULL); - if (IS_ERR_OR_NULL(inode) || is_bad_inode(inode)) { - if (inode && !IS_ERR(inode)) + if (IS_ERR(inode) || is_bad_inode(inode)) { + if (!IS_ERR(inode)) iput(inode); return -ENOENT; } |