diff options
| author | Filipe Manana <fdmanana@suse.com> | 2023-06-08 11:27:38 +0100 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2023-06-19 13:59:38 +0200 |
| commit | ede600e497b1461d06d22a7d17703d9096868bc3 (patch) | |
| tree | e50774d777c94039ade4c1581da4c4333224faab /fs/btrfs/ctree.c | |
| parent | d09c51521f22f9cbdfb1cf63e5c456077c622c84 (diff) | |
| download | linux-ede600e497b1461d06d22a7d17703d9096868bc3.tar.gz linux-ede600e497b1461d06d22a7d17703d9096868bc3.tar.bz2 linux-ede600e497b1461d06d22a7d17703d9096868bc3.zip | |
btrfs: fix extent buffer leak after tree mod log failure at split_node()
At split_node(), if we fail to log the tree mod log copy operation, we
return without unlocking the split extent buffer we just allocated and
without decrementing the reference we own on it. Fix this by unlocking
it and decrementing the ref count before returning.
Fixes: 5de865eebb83 ("Btrfs: fix tree mod logging")
CC: stable@vger.kernel.org # 5.4+
Reviewed-by: Qu Wenruo <wqu@suse.com>
Signed-off-by: Filipe Manana <fdmanana@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/ctree.c')
| -rw-r--r-- | fs/btrfs/ctree.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/btrfs/ctree.c b/fs/btrfs/ctree.c index 7f7f13965fe9..8496535828de 100644 --- a/fs/btrfs/ctree.c +++ b/fs/btrfs/ctree.c @@ -3053,6 +3053,8 @@ static noinline int split_node(struct btrfs_trans_handle *trans, ret = btrfs_tree_mod_log_eb_copy(split, c, 0, mid, c_nritems - mid); if (ret) { + btrfs_tree_unlock(split); + free_extent_buffer(split); btrfs_abort_transaction(trans, ret); return ret; } |