[PATCH] Tests: Fix warning in show-die-info.c

Mark Wielaard mark@klomp.org
Wed Oct 6 21:55:10 GMT 2021


Hi Jan-Benedict,

On Tue, Oct 05, 2021 at 05:36:40PM +0200, Jan-Benedict Glaw wrote:
> 
> My last email had a wrong subject, though the patch was correct.
> Here's a second patch, this time *actally* for tests/show-die-info.c:
> 
> diff --git a/tests/show-die-info.c b/tests/show-die-info.c
> index 34e27a3b..0823cc60 100644
> --- a/tests/show-die-info.c
> +++ b/tests/show-die-info.c
> @@ -97,7 +97,7 @@ handle (Dwarf *dbg, Dwarf_Die *die, int n)
>    printf ("%*s Attrs     :", n * 5, "");
>    for (cnt = 0; cnt < 0xffff; ++cnt)
>      if (dwarf_hasattr (die, cnt))
> -      printf (" %s", dwarf_attr_string (cnt));
> +      printf (" %s", (dwarf_attr_string (cnt)? dwarf_attr_string (cnt): ""));
>    puts ("");
>  
>    if (dwarf_hasattr (die, DW_AT_low_pc) && dwarf_lowpc (die, &addr) == 0)

This can be fixed in a shorter way using dwarf_attr_string ?: "<unknown>".
Which is what I pushed (see attached).

Thanks,

Mark


-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-tests-Handle-dwarf_attr_string-returning-NULL-in-sho.patch
Type: text/x-diff
Size: 1293 bytes
Desc: not available
URL: <https://sourceware.org/pipermail/elfutils-devel/attachments/20211006/ab8f3bec/attachment.bin>


More information about the Elfutils-devel mailing list