Re: [PATCH 05/10 v3] libdw: make dwarf_getalt and dwarf_setalt thread-safe

[Aaron Merey]

Hi Mark,

On Fri, Aug 16, 2024 at 9:45 AM Mark Wielaard <m...@klomp.org> wrote:
>
> On Fri, 2024-08-02 at 19:38 -0400, Aaron Merey wrote:
> > From: Heather McIntyre <h...@rice.edu>
> >
> >       * libdw/dwarf_getalt.c (dwarf_getalt): Add locking.
> >       * libdw/dwarf_setalt.c (dwarf_setalt): Ditto.
> >
> > Signed-off-by: Heather S. McIntyre <h...@rice.edu>
> > Signed-off-by: Aaron Merey <ame...@redhat.com>
> > Signed-off-by: Mark Wielaard <m...@klomp.org>
> >
> > ---
> > v3 changes:
> >
> > Check for NULL argument before acquiring lock. Assign result before
> > releasing lock.  Include dwarf_setalt.c changes that were previously
> > included in patch 5/9 v2.
>
> Cleanups look good. This patch looks good to go.

Thanks, pushed as commit 7c4fcff44ae.

Aaron