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

Mark Wielaard mark@klomp.org
Thu Aug 15 16:16:37 GMT 2024


Hi Aaron,

On Fri, 2024-08-02 at 19:38 -0400, Aaron Merey wrote:
> From: Heather McIntyre <hsm2@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 <hsm2@rice.edu>
> Signed-off-by: Aaron Merey <amerey@redhat.com>
> Signed-off-by: Mark Wielaard <mark@klomp.org>
> 
> ---
> v3 changes:
> Remove unnecessary early return that would require locking to
> check condition.

The code looks good and clean now.

Thanks,

Mark


More information about the Elfutils-devel mailing list