Re: [PATCH] Also find CFI in sections of type SHT_X86_64_UNWIND

2018-11-13 Thread Mark Wielaard
On Sun, 2018-11-11 at 00:28 +0100, Mark Wielaard wrote: > Found another issue while testing with a gcc 8.1 and gold setup I > noticed some self-tests failed because elflint doesn't know about > SHT_X86_64_UNWIND section type and cannot match the PT_GNU_EH_FRAME > segment. The attached patch fixes i

Re: Buildbot failure in Wildebeest Builder on whole buildset

2018-11-13 Thread Mark Wielaard
On Tue, 2018-11-13 at 18:29 +0100, Mark Wielaard wrote: > So we might actually have two bugs. 1) we don't handle incorrectly > aligned compressed sections and 2) we generate such incorrectly > aligned compressed sections ourselves. > > Pondering fixes. The fixes were easy! (Hopefully the buildbot

Re: Buildbot failure in Wildebeest Builder on whole buildset

2018-11-13 Thread Mark Wielaard
On Tue, 2018-11-13 at 16:42 +, build...@builder.wildebeest.org wrote: > The Buildbot has detected a failed build on builder whole buildset > while building elfutils. > Full details are available at: > https://builder.wildebeest.org/buildbot/#builders/3/builds/299 This is disappointing. You

Buildbot failure in Wildebeest Builder on whole buildset

2018-11-13 Thread buildbot
The Buildbot has detected a failed build on builder whole buildset while building elfutils. Full details are available at: https://builder.wildebeest.org/buildbot/#builders/3/builds/299 Buildbot URL: https://builder.wildebeest.org/buildbot/ Worker for this Build: fedora-x86_64 Build Reason:

Re: [PATCH] strip: Also handle gnu compressed debug sections with --reloc-debug-sections

2018-11-13 Thread Mark Wielaard
On Fri, 2018-11-09 at 17:41 +0100, Mark Wielaard wrote: > Check whether a section was gnu compressed and decompress it first > before trying to resolve relocations. Recompress it afterwards. > > This found a bug in elf_compress_gnu which would use the "raw" file > contents even if the user had jus

Re: [PATCH] Handle GNU Build Attribute ELF Notes.

2018-11-13 Thread Mark Wielaard
On Tue, 2018-11-13 at 11:56 +, Nick Clifton wrote: > + /* There might or might not be a pair of addresses in the desc. */ > > FYI - There was an old version of the Watermark specification that only put > one address into the note's desc field. This version was never used in > production bina

Re: [PATCH] Recognize NT_VERSION notes.

2018-11-13 Thread Mark Wielaard
On Mon, 2018-11-12 at 00:08 +0100, Mark Wielaard wrote: > NT_VERSION notes are emitted by the gas .version directive. > They have an empty description and (ab)use the owner name to store > the version data string. I pushed this to master because it is a prerequisite of the GNU Build Attribute ELF