Re: [PATCH] Btrfs: fix extent buffer leak after backref walking

[Jan Schmidt]

On Wed, July 03, 2013 at 08:40 (+0200), Liu Bo wrote:
> commit 47fb091fb787420cd195e66f162737401cce023f(Btrfs: fix unlock after free 
> on rewinded tree blocks)
> takes an extra increment on the reference of allocated dummy extent buffer, 
> so now we
> cannot free this dummy one, and end up with extent buffer leak.
> 
> Signed-off-by: Liu Bo 

Reviewed-by: Jan Schmidt 

> ---
>  fs/btrfs/ctree.c |2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/fs/btrfs/ctree.c b/fs/btrfs/ctree.c
> index 02fae7f..3d790b4 100644
> --- a/fs/btrfs/ctree.c
> +++ b/fs/btrfs/ctree.c
> @@ -1268,12 +1268,12 @@ tree_mod_log_rewind(struct btrfs_fs_info *fs_info, 
> struct extent_buffer *eb,
>   BUG_ON(!eb_rewin);
>   }
>  
> - extent_buffer_get(eb_rewin);
>   btrfs_tree_read_unlock(eb);
>   free_extent_buffer(eb);
>  
>   extent_buffer_get(eb_rewin);
>   btrfs_tree_read_lock(eb_rewin);
> +
>   __tree_mod_log_rewind(eb_rewin, time_seq, tm);
>   WARN_ON(btrfs_header_nritems(eb_rewin) >
>   BTRFS_NODEPTRS_PER_BLOCK(fs_info->tree_root));
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH] Btrfs: fix extent buffer leak after backref walking

[Liu Bo]

commit 47fb091fb787420cd195e66f162737401cce023f(Btrfs: fix unlock after free on 
rewinded tree blocks)
takes an extra increment on the reference of allocated dummy extent buffer, so 
now we
cannot free this dummy one, and end up with extent buffer leak.

Signed-off-by: Liu Bo 
---
 fs/btrfs/ctree.c |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/fs/btrfs/ctree.c b/fs/btrfs/ctree.c
index 02fae7f..3d790b4 100644
--- a/fs/btrfs/ctree.c
+++ b/fs/btrfs/ctree.c
@@ -1268,12 +1268,12 @@ tree_mod_log_rewind(struct btrfs_fs_info *fs_info, 
struct extent_buffer *eb,
BUG_ON(!eb_rewin);
}
 
-   extent_buffer_get(eb_rewin);
btrfs_tree_read_unlock(eb);
free_extent_buffer(eb);
 
extent_buffer_get(eb_rewin);
btrfs_tree_read_lock(eb_rewin);
+
__tree_mod_log_rewind(eb_rewin, time_seq, tm);
WARN_ON(btrfs_header_nritems(eb_rewin) >
BTRFS_NODEPTRS_PER_BLOCK(fs_info->tree_root));
-- 
1.7.7

--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html