diff options
author | Tsutomu Itoh <t-itoh@jp.fujitsu.com> | 2013-04-18 21:04:46 -0400 |
---|---|---|
committer | Josef Bacik <jbacik@fusionio.com> | 2013-05-06 15:55:13 -0400 |
commit | ecc7ada77b5cd1ac525db8f7d4d266e88af66cc7 (patch) | |
tree | b1fbf177d840bd2167d2c3cb50b9e05e4fc8c268 /fs | |
parent | ba1eeaac99ce15063d8bc156e03f8b75aa471647 (diff) |
Btrfs: fix error handling in btrfs_ioctl_send()
fget() returns NULL if error. So, we should check NULL or not.
Signed-off-by: Tsutomu Itoh <t-itoh@jp.fujitsu.com>
Signed-off-by: Josef Bacik <jbacik@fusionio.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/btrfs/send.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/btrfs/send.c b/fs/btrfs/send.c index 158b91984b60..2037fc0efab1 100644 --- a/fs/btrfs/send.c +++ b/fs/btrfs/send.c | |||
@@ -4612,8 +4612,8 @@ long btrfs_ioctl_send(struct file *mnt_file, void __user *arg_) | |||
4612 | sctx->flags = arg->flags; | 4612 | sctx->flags = arg->flags; |
4613 | 4613 | ||
4614 | sctx->send_filp = fget(arg->send_fd); | 4614 | sctx->send_filp = fget(arg->send_fd); |
4615 | if (IS_ERR(sctx->send_filp)) { | 4615 | if (!sctx->send_filp) { |
4616 | ret = PTR_ERR(sctx->send_filp); | 4616 | ret = -EBADF; |
4617 | goto out; | 4617 | goto out; |
4618 | } | 4618 | } |
4619 | 4619 | ||