diff options
author | Dan Carpenter <error27@gmail.com> | 2010-05-29 11:47:24 +0200 |
---|---|---|
committer | Chris Mason <chris.mason@oracle.com> | 2010-06-11 21:57:37 +0200 |
commit | cf1e99a4e0daa4a5623cd71108509823b9ff2d30 (patch) | |
tree | e2e613940e9d4449be976a5a09e1c2a72bb45a83 | |
parent | Btrfs: btrfs_read_fs_root_no_name() returns ERR_PTRs (diff) | |
download | linux-cf1e99a4e0daa4a5623cd71108509823b9ff2d30.tar.xz linux-cf1e99a4e0daa4a5623cd71108509823b9ff2d30.zip |
Btrfs: btrfs_lookup_dir_item() can return ERR_PTR
btrfs_lookup_dir_item() can return either ERR_PTRs or null.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Chris Mason <chris.mason@oracle.com>
-rw-r--r-- | fs/btrfs/ioctl.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/btrfs/ioctl.c b/fs/btrfs/ioctl.c index 9f9a1d9607a7..4dbaf89b1337 100644 --- a/fs/btrfs/ioctl.c +++ b/fs/btrfs/ioctl.c @@ -1845,7 +1845,7 @@ static long btrfs_ioctl_default_subvol(struct file *file, void __user *argp) dir_id = btrfs_super_root_dir(&root->fs_info->super_copy); di = btrfs_lookup_dir_item(trans, root->fs_info->tree_root, path, dir_id, "default", 7, 1); - if (!di) { + if (IS_ERR_OR_NULL(di)) { btrfs_free_path(path); btrfs_end_transaction(trans, root); printk(KERN_ERR "Umm, you don't have the default dir item, " |