commit 57a5a882031dba5cb7bc7ebc955b897498365fe2
author Dan Carpenter <dan.carpenter@oracle.com> Mon Jul 30 02:15:43 2012 -0600
committer Chris Mason <chris.mason@oracle.com> Tue Aug 28 16:53:25 2012 -0400
tree cae85b46388641513146af860f0140d26811ae57
parent 5986802c2fcc754040bb7ed95f30bb16c4a843b7

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>

fs/btrfs/qgroup.c

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 @@
 	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);