diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2015-11-10 09:10:03 (GMT) |
---|---|---|
committer | Chris Mason <clm@fb.com> | 2015-11-25 13:19:50 (GMT) |
commit | 89b6c8d1e4a178a347b94f339b959f02710e7060 (patch) | |
tree | 77bcfcb9063302849cdb3d5be95718585e52ba1d /fs/btrfs | |
parent | 9dcbeed4d7e11e1dcf5e55475de3754f0855d1c2 (diff) | |
download | linux-89b6c8d1e4a178a347b94f339b959f02710e7060.tar.xz |
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>
Diffstat (limited to 'fs/btrfs')
-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 c8c3d70..8b72b00 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) } root = btrfs_alloc_dummy_root(); - if (!root) + if (IS_ERR(root)) { + ret = PTR_ERR(root); goto out; + } root->fs_info = btrfs_alloc_dummy_fs_info(); if (!root->fs_info) |