☠ Buildbot (Sourceware): debugedit - failed test (failure) test (failure) (main)

[Mark Wielaard]

Hi,

On Mon, Sep 07, 2026 at 06:25:33PM +0200, Mark Wielaard wrote:
> So the new sanity checks fail (only) on opensuse leap and tumbleweed.
> A quick test showed it is only this one:
> 
> commit e6833558191eecc011df9c7e8e734e10ac79cec1
> Author: Mark Wielaard <mark@klomp.org>
> Date:   Tue Sep 1 13:52:02 2026 +0200
> 
>     debugedit: Add CU sanity check to edit_info
>     
>     We rely on the .debuginfo sections to be valid DWARF, for example
> to
>     use embedded string (DW_FORM_string). Add a sanity check that each
> CU
>     should end in a zero DIE ('\0').
>     
>             * tools/debugedit.c (edit_info): Check CU ends with zero
> DIE.
> 
> All other new sanity checks work as expected.
> 
> Still investigating why this one doesn't.

So a strict reading of DWARF says a zero CU DIE entry is just for
padding. And at least binutils gas doesn't output any. That this only
is a problem on opensuse seems to be because it always includes the
crt entry with debuginfo produced with gas. But it could happen
everywhere. So the above patch is indeed wrong. I will submit
something that works around this issue.

Cheers,

Mark