summaryrefslogtreecommitdiffstats
path: root/fs
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2012-04-18 09:59:03 +0300
committerDavid Sterba <dsterba@suse.cz>2012-04-18 19:22:31 +0200
commit253beebd5a255e07d6a8b65515491f33664e82a2 (patch)
tree9bc6b5f567b7683df4b2a8d46ecdc6d8dd139d1e /fs
parent5cf1ab56133ad7b712673c071b439d4a555a2d1e (diff)
downloadop-kernel-dev-253beebd5a255e07d6a8b65515491f33664e82a2.zip
op-kernel-dev-253beebd5a255e07d6a8b65515491f33664e82a2.tar.gz
Btrfs: double unlock bug in error handling
The caller expects this function to return with the lock held and releases it immediately on error. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Diffstat (limited to 'fs')
-rw-r--r--fs/btrfs/extent-tree.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/btrfs/extent-tree.c b/fs/btrfs/extent-tree.c
index 2b35f8d..a0bb9dc 100644
--- a/fs/btrfs/extent-tree.c
+++ b/fs/btrfs/extent-tree.c
@@ -2301,6 +2301,7 @@ static noinline int run_clustered_refs(struct btrfs_trans_handle *trans,
if (ret) {
printk(KERN_DEBUG "btrfs: run_delayed_extent_op returned %d\n", ret);
+ spin_lock(&delayed_refs->lock);
return ret;
}
@@ -2331,6 +2332,7 @@ static noinline int run_clustered_refs(struct btrfs_trans_handle *trans,
if (ret) {
printk(KERN_DEBUG "btrfs: run_one_delayed_ref returned %d\n", ret);
+ spin_lock(&delayed_refs->lock);
return ret;
}
OpenPOWER on IntegriCloud