diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-04-18 09:59:03 +0300 |
|---|---|---|
| committer | David Sterba <dsterba@suse.cz> | 2012-04-18 19:22:31 +0200 |
| commit | 253beebd5a255e07d6a8b65515491f33664e82a2 (patch) | |
| tree | 9bc6b5f567b7683df4b2a8d46ecdc6d8dd139d1e | |
| parent | 5cf1ab56133ad7b712673c071b439d4a555a2d1e (diff) | |
| download | linux-stable-253beebd5a255e07d6a8b65515491f33664e82a2.tar.gz linux-stable-253beebd5a255e07d6a8b65515491f33664e82a2.tar.bz2 linux-stable-253beebd5a255e07d6a8b65515491f33664e82a2.zip | |
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>
| -rw-r--r-- | fs/btrfs/extent-tree.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/btrfs/extent-tree.c b/fs/btrfs/extent-tree.c index 2b35f8d14bb9..a0bb9dcd3c36 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; } |