diff options
| author | Tsutomu Itoh <t-itoh@jp.fujitsu.com> | 2011-01-24 03:57:10 +0300 |
|---|---|---|
| committer | Chris Mason <chris.mason@oracle.com> | 2011-01-29 00:40:37 +0300 |
| commit | abd30bb0af9d4671506502278e8631bed9e3c35c (patch) | |
| tree | 73e00f8712c8e4460647dd889cc32503fb56c21b /fs/btrfs | |
| parent | 3612b49598c303cfb22a4b609427f829828e2427 (diff) | |
| download | linux-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')
| -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 04b4fb9144a9..12dabe28cf54 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; |