Hi Aaron,

On Sun, 2025-01-19 at 22:20 -0500, Aaron Merey wrote:
>       * libdw/dwarf_begin_elf.c (dwarf_begin_elf): Initialize
>       eu_search_trees.

I see the same are already eu_search_tree_fini in dwarf_end. So my only
question is how this worked before?

Whatever the answer is though, this looks good.

Thanks,

Mark

> Signed-off-by: Aaron Merey <ame...@redhat.com>
> 
> ---
>  libdw/dwarf_begin_elf.c | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/libdw/dwarf_begin_elf.c b/libdw/dwarf_begin_elf.c
> index e4826dda..c8292913 100644
> --- a/libdw/dwarf_begin_elf.c
> +++ b/libdw/dwarf_begin_elf.c
> @@ -580,6 +580,12 @@ dwarf_begin_elf (Elf *elf, Dwarf_Cmd cmd, Elf_Scn 
> *scngrp)
>        return NULL;
>      }
>    rwlock_init (result->dwarf_lock);
> +  eu_search_tree_init (&result->cu_tree);
> +  eu_search_tree_init (&result->tu_tree);
> +  eu_search_tree_init (&result->split_tree);
> +  eu_search_tree_init (&result->macro_ops_tree);
> +  eu_search_tree_init (&result->files_lines_tree);
> +
>    result->mem_stacks = 0;
>    result->mem_tails = NULL;
>  

Reply via email to