diff options
author | Tsutomu Itoh <t-itoh@jp.fujitsu.com> | 2011-01-24 00:57:10 (GMT) |
---|---|---|
committer | Chris Mason <chris.mason@oracle.com> | 2011-01-28 21:40:37 (GMT) |
commit | abd30bb0af9d4671506502278e8631bed9e3c35c (patch) | |
tree | 73e00f8712c8e4460647dd889cc32503fb56c21b /fs/btrfs/ioctl.c | |
parent | 3612b49598c303cfb22a4b609427f829828e2427 (diff) | |
download | linux-fsl-qoriq-abd30bb0af9d4671506502278e8631bed9e3c35c.tar.xz |
btrfs: check return value of btrfs_start_ioctl_transaction() properly
btrfs_start_ioctl_transaction() returns ERR_PTR(), not NULL.
So, it is necessary to use IS_ERR() to check the return value.
Signed-off-by: Tsutomu Itoh <t-itoh@jp.fujitsu.com>
Signed-off-by: Chris Mason <chris.mason@oracle.com>
Diffstat (limited to 'fs/btrfs/ioctl.c')
-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 04b4fb9..12dabe2 100644 --- a/fs/btrfs/ioctl.c +++ b/fs/btrfs/ioctl.c @@ -2085,7 +2085,7 @@ static long btrfs_ioctl_trans_start(struct file *file) ret = -ENOMEM; trans = btrfs_start_ioctl_transaction(root, 0); - if (!trans) + if (IS_ERR(trans)) goto out_drop; file->private_data = trans; |