diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-07-30 08:15:43 (GMT) |
|---|---|---|
| committer | Chris Mason <chris.mason@oracle.com> | 2012-08-28 20:53:25 (GMT) |
| commit | 57a5a882031dba5cb7bc7ebc955b897498365fe2 (patch) | |
| tree | cae85b46388641513146af860f0140d26811ae57 /fs/btrfs | |
| parent | 5986802c2fcc754040bb7ed95f30bb16c4a843b7 (diff) | |
| download | linux-57a5a882031dba5cb7bc7ebc955b897498365fe2.tar.xz | |
Btrfs: checking for NULL instead of IS_ERR
add_qgroup_rb() never returns NULL, only error pointers.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Diffstat (limited to 'fs/btrfs')
| -rw-r--r-- | fs/btrfs/qgroup.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/btrfs/qgroup.c b/fs/btrfs/qgroup.c index 229ef89..38b42e7 100644 --- a/fs/btrfs/qgroup.c +++ b/fs/btrfs/qgroup.c @@ -1364,8 +1364,10 @@ int btrfs_qgroup_inherit(struct btrfs_trans_handle *trans, spin_lock(&fs_info->qgroup_lock); dstgroup = add_qgroup_rb(fs_info, objectid); - if (!dstgroup) + if (IS_ERR(dstgroup)) { + ret = PTR_ERR(dstgroup); goto unlock; + } if (srcid) { srcgroup = find_qgroup_rb(fs_info, srcid); |