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

Aaron Merey amerey@redhat.com
Wed Aug 21 00:31:30 GMT 2024


Hi Mark,

On Fri, Aug 16, 2024 at 9:45 AM Mark Wielaard <mark@klomp.org> wrote:
>
> On Fri, 2024-08-02 at 19:38 -0400, Aaron Merey wrote:
> > From: Heather McIntyre <hsm2@rice.edu>
> >
> >       * libdw/dwarf_getalt.c (dwarf_getalt): Add locking.
> >       * libdw/dwarf_setalt.c (dwarf_setalt): Ditto.
> >
> > Signed-off-by: Heather S. McIntyre <hsm2@rice.edu>
> > Signed-off-by: Aaron Merey <amerey@redhat.com>
> > Signed-off-by: Mark Wielaard <mark@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



More information about the Elfutils-devel mailing list