https://sourceware.org/bugzilla/show_bug.cgi?id=30980

--- Comment #6 from Mark Wielaard <mark at klomp dot org> ---
Thanks, that seems to be the correct thing to do here.

Note that there are a couple more asserts on this code path. In particular in
__libdwfl_find_elf_build_id there is what apparently is the root of the
confusion here:

  // MOD->E_TYPE is zero here.
  assert (ehdr->e_type != ET_REL || mod != NULL);

Which passes because mod != NULL. But probably should have been a red flag.
This is my fault, I refactored this code and kept these asserts without fully
understanding why there were (not) needed.

We should go over all asserts in the library code and rewrite/remove them. It
is never a good thing to trigger an abort in library code.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

Reply via email to