Re: [PATCH 02/10 v3] libelf: Fix deadlock in elf_cntl

[Aaron Merey]

Hi Mark,

On Thu, Aug 15, 2024 at 12:16 PM Mark Wielaard <m...@klomp.org> wrote:
>
> On Fri, 2024-08-02 at 19:38 -0400, Aaron Merey wrote:
> > From: Heather McIntyre <h...@rice.edu>
> >
> >         * libelf/elf_cntl.c (elf_cntl): Move rwlock_wrlock, rwlock_unlock,
> >         inside case switch statements.  Remove unnecessary early return.
> >
> > 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:
> > Remove unnecessary early return that would require locking to
> > check condition.
>
> The code looks good and clean now.

Thanks, pushed as commit 97b72c00603.

Aaron