diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2015-11-10 04:10:03 -0500 |
---|---|---|
committer | Chris Mason <clm@fb.com> | 2015-11-25 08:19:50 -0500 |
commit | 89b6c8d1e4a178a347b94f339b959f02710e7060 (patch) | |
tree | 77bcfcb9063302849cdb3d5be95718585e52ba1d | |
parent | 9dcbeed4d7e11e1dcf5e55475de3754f0855d1c2 (diff) |
Btrfs: tests: checking for NULL instead of IS_ERR()
btrfs_alloc_dummy_root() return an error pointer on failure, it never
returns NULL.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: Chris Mason <clm@fb.com>
-rw-r--r-- | fs/btrfs/tests/free-space-tests.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/btrfs/tests/free-space-tests.c b/fs/btrfs/tests/free-space-tests.c index c8c3d70c31ff..8b72b005bfb9 100644 --- a/fs/btrfs/tests/free-space-tests.c +++ b/fs/btrfs/tests/free-space-tests.c | |||
@@ -898,8 +898,10 @@ int btrfs_test_free_space_cache(void) | |||
898 | } | 898 | } |
899 | 899 | ||
900 | root = btrfs_alloc_dummy_root(); | 900 | root = btrfs_alloc_dummy_root(); |
901 | if (!root) | 901 | if (IS_ERR(root)) { |
902 | ret = PTR_ERR(root); | ||
902 | goto out; | 903 | goto out; |
904 | } | ||
903 | 905 | ||
904 | root->fs_info = btrfs_alloc_dummy_fs_info(); | 906 | root->fs_info = btrfs_alloc_dummy_fs_info(); |
905 | if (!root->fs_info) | 907 | if (!root->fs_info) |