diff options
author | 2011-03-23 08:14:16 +0000 | |
---|---|---|
committer | 2011-03-28 05:37:35 -0400 | |
commit | db5b493ac78e46c7b6bad22cd25d8041564cd8ea (patch) | |
tree | 18f484c3d8194b80165e5400831931a3ac5c58ba /fs/btrfs/ioctl.c | |
parent | Btrfs: add initial tracepoint support for btrfs (diff) | |
download | linux-db5b493ac78e46c7b6bad22cd25d8041564cd8ea.tar.gz linux-db5b493ac78e46c7b6bad22cd25d8041564cd8ea.tar.bz2 linux-db5b493ac78e46c7b6bad22cd25d8041564cd8ea.zip |
Btrfs: cleanup some BUG_ON()
This patch changes some BUG_ON() to the error return.
(but, most callers still use BUG_ON())
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 | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/fs/btrfs/ioctl.c b/fs/btrfs/ioctl.c index ad9b8c0e930b..88d3cb2eaf75 100644 --- a/fs/btrfs/ioctl.c +++ b/fs/btrfs/ioctl.c @@ -2350,12 +2350,15 @@ static noinline long btrfs_ioctl_start_sync(struct file *file, void __user *argp struct btrfs_root *root = BTRFS_I(file->f_dentry->d_inode)->root; struct btrfs_trans_handle *trans; u64 transid; + int ret; trans = btrfs_start_transaction(root, 0); if (IS_ERR(trans)) return PTR_ERR(trans); transid = trans->transid; - btrfs_commit_transaction_async(trans, root, 0); + ret = btrfs_commit_transaction_async(trans, root, 0); + if (ret) + return ret; if (argp) if (copy_to_user(argp, &transid, sizeof(transid))) |