[PATCH 1/2] Initialize type_offset of fake_cu

[Mark Wielaard]

On Wed, 2017-09-06 at 12:38 +0200, Ulf Hermann wrote:
> On 05/09/2017 06:28 PM, Ulf Hermann wrote:
> > Signed-off-by: Ulf Hermann <ulf.hermann@qt.io>
> > ---
> >  libdw/ChangeLog         | 4 ++++
> >  libdw/dwarf_getmacros.c | 1 +
> >  2 files changed, 5 insertions(+)
> > [...]
> 
> It seems we missed this one, but it fixes a bug. Can we apply it?
> PATCH 2/2 is not actually related.

I assume you mean this patch:

> diff --git a/libdw/dwarf_getmacros.c b/libdw/dwarf_getmacros.c
> index eb50508..9e2a4a1 100644
> --- a/libdw/dwarf_getmacros.c
> +++ b/libdw/dwarf_getmacros.c
> @@ -361,6 +361,7 @@ read_macros (Dwarf *dbg, int sec_index,
>         .offset_size = table->is_64bit ? 8 : 4,
>         .startp = (void *) startp + offset,
>         .endp = (void *) endp,
> +       .type_offset = 0,
>        };
>  
>        Dwarf_Attribute *attributes;

That really shouldn't be necessary. In a named struct field initializer
omitted elements are initialized to zero. Are you sure it fixes a real
bug? In that case it might be a compiler bug instead (which we would
want to work around, but then lets make sure to report it and add a
comment).

Cheers,

Mark