[PATCH 6/7] libdwfl: Return failure from dwfl_standard_find_debuginfo for NULL module.

GCC10 -fanalyzer plus -flto notices that some functions called by dwfl_standard_find_debuginfo check that the given module isn't NULL, but others expect it to be non-NULL. Just return a failure immediately when a NULL mod is passed to dwfl_standard_find_debuginfo. Signed-off-by: Mark Wie

[PATCH 7/7] libcpu: Free unused new bitfield on error in i386_parse.y new_bitfield.

GCC10 -fanalyzer detected we didn't free the newly created bitfield on error. Make sure to free it before returning. Signed-off-by: Mark Wielaard --- libcpu/ChangeLog| 4 libcpu/i386_parse.y | 1 + 2 files changed, 5 insertions(+) diff --git a/libcpu/ChangeLog b/libcpu/ChangeLog

Re: [PATCH 1/2] libdw: Use correct CU to resolve file names in dwarf_decl_file.

On Fri, 2020-05-08 at 18:50 +0200, Mark Wielaard wrote: > dwarf_decl_file uses dwarf_attr_integrate to get the DW_AT_decl_file > attribute. This means the attribute might come from a different DIE > in a different CU. If so, we need to use the CU associated with the > attribute, not

Re: [PATCH 2/2] libdw: Skip imported compiler_units in libdw_visit_scopes walking DIE tree

On Fri, 2020-05-08 at 18:50 +0200, Mark Wielaard wrote: > Some gcc -flto versions imported other top-level compile units, > skip those. Otherwise we'll visit various DIE trees multiple times. > > Note in the testcase that with newer GCC versions function foo is > fully inl

Re: Some more GCC10 -fanalyzer inspired patches

On Sun, 2020-05-10 at 21:53 +0200, Mark Wielaard wrote: > I did another build with the final GCC10 and -fanalyzer. The > -Wanalyzer-use-of-uninitialized-value option was removed, which caused > a lot of false positives. Without those it was easier to identify some > real issues.

[COMMITTED] size: Also obey radix printing for bsd format.

eu-size already prints according to the given radix for sysv format, but not for the bsd format. Fix that. Reported-by: Ulrich Drepper Signed-off-by: Mark Wielaard --- src/ChangeLog | 4 src/size.c| 7 +-- 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/src

Re: Range lists, zero-length functions, linker gc

Hi, On Sun, May 31, 2020 at 11:55:06AM -0700, Fangrui Song via Elfutils-devel wrote: > what linkers should do regarding relocations referencing dropped > functions (due to section group rules, --gc-sections, /DISCARD/, > etc) in .debug_* > > As an example: > > __attribute__((section(".text.x"))

Re: Range lists, zero-length functions, linker gc

Hi, On Sun, May 31, 2020 at 01:47:38PM -0700, Fangrui Song via Elfutils-devel wrote: > On 2020-05-31, Mark Wielaard wrote: > > I think this is a producer problem. If a (code) section can be totally > > dropped then the associated (.debug) sections should have been > > generat

Re: Range lists, zero-length functions, linker gc

Hi, On Sun, May 31, 2020 at 01:49:12PM -0700, David Blaikie wrote: > On Sun, May 31, 2020 at 1:41 PM Mark Wielaard wrote: > > On Sun, May 31, 2020 at 11:55:06AM -0700, Fangrui Song via Elfutils-devel > > wrote: > > > I am eager to know what you think > > &g

Re: Range lists, zero-length functions, linker gc

Hi, On Sun, May 31, 2020 at 03:36:02PM -0700, David Blaikie wrote: > On Sun, May 31, 2020 at 3:30 PM Mark Wielaard wrote: > > On Sun, May 31, 2020 at 01:49:12PM -0700, David Blaikie wrote: > > > That's probably not practical for at least some users - the > > &g

Re: Range lists, zero-length functions, linker gc

Hi, On Mon, 2020-06-01 at 13:18 -0700, David Blaikie wrote: > On Mon, Jun 1, 2020 at 2:31 AM Mark Wielaard wrote: > > Each skeleton compilation unit has a DW_AT_dwo_name attribute which > > indicates the .dwo file where the split unit sections can be found. It > > actually

Re: location list

Hi, On Tue, 2020-06-02 at 14:18 +, Sasha Da Rocha Pinheiro wrote: > I am trying to parse a location list given as an sec_offset. > How do I get this offset value that points to .debug_loc so I can > call dwarf_getlocations()? > Should I pass this offset as the second parameter of this call?

[COMMITTED] libelf: Sync elf.h from glibc.

There is now finally a PT_GNU_PROPERTY define. Also EM_ARC_COMPACT2 got renamed to EM_ARCV2. Signed-off-by: Mark Wielaard --- libdwelf/ChangeLog| 5 ++ libdwelf/dwelf_elf_e_machine_string.c | 4 +- libebl/ChangeLog | 5 ++ libebl

Re: location list

Hi Sasha, On Sat, 2020-06-06 at 00:30 +, Sasha Da Rocha Pinheiro wrote: > As you can see the following variables have distinct locations: > [81] variable abbrev: 5 >name (string) "a" >decl_file(data1) sasha.c (1)

[PATCH] nm: Explicitly print weak 'V' or 'T' and common 'C' symbols.

NIQUE for GNU_UNIQUE to make it fit 6 chars, like other binding names in sysv format. https://sourceware.org/bugzilla/show_bug.cgi?id=25227 Signed-off-by: Mark Wielaard --- src/ChangeLog| 10 +++ src/nm.c | 59 ++ tests/ChangeLog | 9 +++ tests

[PATCH] nm: Handle corrupt symbol name table.

We try to sort symbol by name (when neither -n nor -p are given). This could crash if the symbol name table was corrupt. Use elf_strptr to get the symbol name and use the empty string in case a name couldn't be found. Signed-off-by: Mark Wielaard --- src/ChangeLog | 8 src

Re: [PATCH] nm: Explicitly print weak 'V' or 'T' and common 'C' symbols.

On Sun, 2020-06-07 at 01:12 +0200, Mark Wielaard wrote: > Mimic binutils nm for bsd and posix formats which uses 'V' for weak > symbols, 'C' for common symbols and 'T' for weak functions. Also fix > some formatting issues. Don't print undefined address

Re: [PATCH] nm: Handle corrupt symbol name table.

On Sun, 2020-06-07 at 16:51 +0200, Mark Wielaard wrote: > We try to sort symbol by name (when neither -n nor -p are given). > This could crash if the symbol name table was corrupt. Use elf_strptr > to get the symbol name and use the empty string in case a name > couldn't be

Re: location list

Hi Sasha, On Tue, 2020-06-09 at 16:38 +, Sasha Da Rocha Pinheiro via Elfutils-devel wrote: > I am now trying to design the changes needed to be done in Dyninst. > So far we have only used the functions dwarf_* under libdw. > What I understood is that libdw is kinda divided in subsets of functi

[COMMITTED] libebl: Remove Ebl struct size check and MODVERSION string.

ently indicates any error). Signed-off-by: Mark Wielaard --- ChangeLog | 4 backends/ChangeLog | 20 backends/aarch64_init.c | 11 +++ backends/alpha_init.c | 11 +++ backends/arm_init.c | 11 +++ backends/bpf_init.c

[COMMITTED] Prepare for 0.180

Set version to 0.180. Update NEWS and elfutils.spec.in. Set copyright year in printversion. Regenerate po/*.po files. Update .gitignore. Signed-off-by: Mark Wielaard --- .gitignore | 27 - ChangeLog | 6 ++ NEWS| 14 +++ config/ChangeLog

Re: Buildbot failure in Wildebeest Builder on whole buildset

On Thu, Jun 11, 2020 at 04:00:38PM +, 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/1/builds/546 > > Buildbot URL:

[COMMITTED] tests: Add GCOV flags when necessary.

The nlist-test has a special compile rule, make sure it gets the GCOV flags when configuring with --enable-gcov. Signed-off-by: Mark Wielaard --- tests/ChangeLog | 4 tests/Makefile.am | 8 +++- 2 files changed, 11 insertions(+), 1 deletion(-) diff --git a/tests/ChangeLog b/tests

elfutils 0.180 released

ysconfig: add /var/lib/pulp as default search path Mark Wielaard (27): config: chmod uploaded dir and files to make them readable. libdwfl: Initialize bits to NULL in dwfl_standard_find_debuginfo for LTO. libcpu: Replace assert with goto invalid_op for bad prefix. elflint: Allow SHF_EXCLU

Re: elfutils 0.180 released

Hi Martin, On Thu, 2020-06-11 at 20:51 +0200, Martin Liška wrote: > On 6/11/20 8:31 PM, Martin Liška wrote: > > On 6/11/20 7:48 PM, Mark Wielaard wrote: > > > ELFUTILS 0.180 -http://elfutils.org/ > > > > Thank you for the release. I'm seeing a

Re: [PATCH] elfutils/configure.ac: use $READELF, not readelf

Hi Sergei, On Sun, 2020-06-14 at 20:28 +0100, Sergei Trofimovich wrote: > Allow user to specify own readelf. Use detected readelf, > not 'readelf'. You are absolutely right. We have an explicit AC_CHECK_TOOL for readelf, so we should use the result whenever we use READELF. Added ChangeLog entry

[COMMITTED] tests: Use #/usr/bin/env bash in scripts that use bash

Some systems don't have bash in /bin. Signed-off-by: Mark Wielaard --- config/ChangeLog | 4 +++ config/upload-release.sh | 2 +- tests/ChangeLog | 30 +++ tests/coverage.sh

[PATCH 05/10] libdwfl: Flag an error if CIE return_address_register is invalid.

If the CIE return address register is invalid (unknown) for the architecture immediately flag an error and return. Signed-off-by: Mark Wielaard --- libdwfl/ChangeLog | 5 + libdwfl/frame_unwind.c | 6 +- 2 files changed, 10 insertions(+), 1 deletion(-) diff --git a/libdwfl

[PATCH 10/10] debuginfod: Make sure handle_data can be allocated and is always freed.

When allocating handle_data we should check for out of memory failures. Also when the allocation has succeeded make sure we always clean up by going to out1 on any future errors. So move the curl_multi_init call earlier, because that goes to out0 on failure. Signed-off-by: Mark Wielaard

[PATCH 02/10] libdw: Add missing FALLTHROUGH in execute_cfi.

Signed-off-by: Mark Wielaard --- libdw/ChangeLog | 4 libdw/cfi.c | 1 + 2 files changed, 5 insertions(+) diff --git a/libdw/ChangeLog b/libdw/ChangeLog index c75b0958..72cd5003 100644 --- a/libdw/ChangeLog +++ b/libdw/ChangeLog @@ -1,3 +1,7 @@ +2020-06-16 Mark Wielaard

[PATCH 04/10] libdwfl: When we find a compressed image, use that, don't search for others

We try to find a compressed vmlinux image ending with either .gz, bz2 or xz. Stop searching if we find one. Otherwise we will leak a file descriptor for an earlier one we opened. Signed-off-by: Mark Wielaard --- libdwfl/ChangeLog | 5 + libdwfl/linux-kernel-modules.c | 2 +- 2

[PATCH 03/10] elflint: Explicitly check and ignore elf_compress error.

se in the code we do explicitly check for errors. Signed-off-by: Mark Wielaard --- src/ChangeLog | 5 + src/elflint.c | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/src/ChangeLog b/src/ChangeLog index 512d7b54..e78bc358 100644 --- a/src/ChangeLog +++ b/src/ChangeLog @@

[PATCH 07/10] debuginfod: Handle not being able to fopen interval_path.

Although we check for and/or create the interval_path right before, there is still a possibility that the fopen call fails. Handle that as if the file is unreadable. Signed-off-by: Mark Wielaard --- debuginfod/ChangeLog | 5 + debuginfod/debuginfod-client.c | 9 +++-- 2 files

[PATCH 08/10] debuginfod: Make sure suffix can place zero terminator when copying filename

@@ +2020-06-16 Mark Wielaard + + * debuginfod-client.c (debuginfod_query_server): Increase suffix + array and prepare having to escape 1 character with 2. + 2020-06-16 Mark Wielaard * debuginfod-client.c (debuginfod_clean_cache): Handle failing diff --git a/debuginfod

[PATCH 09/10] debuginfod: Fix build_id hexadecimal length check.

end of the build_id_bytes array. Signed-off-by: Mark Wielaard --- debuginfod/ChangeLog | 5 + debuginfod/debuginfod-client.c | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/debuginfod/ChangeLog b/debuginfod/ChangeLog index 9ff2e111..d6bbfac8 100644 --- a

[PATCH 01/10] backends: Make the reloc_nametable zero element an one char array.

We are using the reloc_nametable zero element as an char array. So make that element an actual array (we are actually after one of the next string arrays in the table). Signed-off-by: Mark Wielaard --- backends/ChangeLog | 6 ++ backends/common-reloc.c | 6 +++--- 2 files changed, 9

[PATCH 06/10] ar, ranlib: Call fchown before fchmod, explicitly check and ignore errors.

Also explicitly test and then ignore any errors for chown. We used to do some asm trick, but that confuses some static analyzers (and it is somewhat unreadable). Also split out the giant if statements to make them a little bit more understandable. Signed-off-by: Mark Wielaard --- src/ChangeLog

Re: [PATCH 01/10] backends: Make the reloc_nametable zero element an one char array.

On Wed, 2020-06-17 at 00:25 +0200, Mark Wielaard wrote: > We are using the reloc_nametable zero element as an char array. > So make that element an actual array (we are actually after one > of the next string arrays in the table). Pushed to master.

Re: Range lists, zero-length functions, linker gc

Hi, On Tue, 2020-06-02 at 11:06 -0700, David Blaikie via Elfutils-devel wrote: > > I do think combining Split DWARF and LTO might not be the best > > solution. When doing LTO you probably want something like GCC Early > > Debug, which is like Split DWARF, but different, because the Early > > Debug

Re: [PATCH 02/10] libdw: Add missing FALLTHROUGH in execute_cfi.

Pushed to master.

Re: [PATCH 03/10] elflint: Explicitly check and ignore elf_compress error.

On Wed, 2020-06-17 at 00:25 +0200, Mark Wielaard wrote: > We just want to make sure that any section data is decompressed before > use, if the section was already decompressed that is fine, so just ignore > any errors. The make this more clear, explicitly check for errors, then > don&#

Re: [PATCH 04/10] libdwfl: When we find a compressed image, use that, don't search for others

On Wed, 2020-06-17 at 00:25 +0200, Mark Wielaard wrote: > We try to find a compressed vmlinux image ending with either .gz, bz2 or > xz. Stop searching if we find one. Otherwise we will leak a file descriptor > for an earlier one we opened. Pushed to master

Re: [PATCH 05/10] libdwfl: Flag an error if CIE return_address_register is invalid.

On Wed, 2020-06-17 at 00:25 +0200, Mark Wielaard wrote: > If the CIE return address register is invalid (unknown) for the > architecture immediately flag an error and return. Pushed to master.

Re: [PATCH 06/10] ar, ranlib: Call fchown before fchmod, explicitly check and ignore errors.

On Wed, 2020-06-17 at 00:25 +0200, Mark Wielaard wrote: > In ar and ranlib we don't mind if the fchown call fails (it normally > would, then the file simply gets own by the current user). We used to > call fchown before fchmod, but that might ignore (or reset) some mode > flag

Re: Tombstone values in debug sections (was: Range lists, zero-length functions, linker gc)

Hi, On Tue, 2020-06-09 at 13:24 -0700, Fangrui Song via Elfutils-devel wrote: > I want to revive the thread, but focus on whether a tombstone value > (-1/-2) in .debug_* can cause trouble to various DWARF consumers (gdb, > debug related tools in elfutils and other utilities I don't know about). >

Re: Question on how to copy an elf file and edit the .preinit_array section

Hi, On Thu, 2020-06-11 at 15:25 +0100, Anastasios Andronidis via Elfutils- devel wrote: > Could someone please help me with this: > https://stackoverflow.com/questions/62293512/inject-a-preinit-array-section-in-an-elf-file-with-libelf > > I'm trying to do some runtime manipulation of an elf fil

Re: Difference between dwarf_getscopes and dwarf_getscopes_die

On Mon, 2020-06-15 at 09:50 -0700, Josh Stone via Elfutils-devel wrote: > On 6/13/20 10:34 AM, Milian Wolff wrote: > > can someone explain me the difference between dwarf_getscopes and > > dwarf_getscopes_die? Ideally, this should then be added to the > > documentation > > too. > > dwarf_getscop

Re: Can dwarf_getscopes{,_die} performance be improved?

On Mon, 2020-06-15 at 09:54 -0700, Josh Stone via Elfutils-devel wrote: > On 6/13/20 10:40 AM, Milian Wolff wrote: > > Has anyone an idea on how to to post-process the DWARF data to > > optimize the > > lookup of inlined frames? > > SystemTap implements its own cache for repeated lookups -- see >

Re: Buildbot failure in Wildebeest Builder on whole buildset

Hi, On Fri, 2020-06-19 at 23:20 +, 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/16/builds/343 > > Buildbot URL: ht

Re: [PATCH 07/10] debuginfod: Handle not being able to fopen interval_path.

Hi, On Wed, 2020-06-17 at 00:25 +0200, Mark Wielaard wrote: > Although we check for and/or create the interval_path right before, > there is still a possibility that the fopen call fails. Handle that > as if the file is unreadable. On Wed, 2020-06-17 at 00:25 +0200, Mark Wielaard wr

Re: Range lists, zero-length functions, linker gc

Hi David, On Fri, 2020-06-19 at 17:46 -0700, David Blaikie via Elfutils-devel wrote: > On Fri, Jun 19, 2020 at 5:00 AM Mark Wielaard wrote: > > I think that is kind of the point of Early Debug. Only use DWARF (at > > first) for address/range-less data like types and program sc

Re: location list

Hi Sasha, On Tue, 2020-06-23 at 16:34 +, Sasha Da Rocha Pinheiro wrote: > Since we are now using not only executables and .so, but ".o" files > too, I'm trying to decide if I can use the same functions to all of > them, like the code you pointed out to deal with ".o". Would that > work for EXE

Re: Can dwarf_getscopes{,_die} performance be improved?

Hi Milian, On Mon, 2020-06-22 at 10:29 +0200, Milian Wolff wrote: > On Montag, 15. Juni 2020 18:54:41 CEST Josh Stone wrote: > > On 6/13/20 10:40 AM, Milian Wolff wrote: > > > Has anyone an idea on how to to post-process the DWARF data to optimize > > > the > > > lookup of inlined frames? > > > >

Re: Questions regarding editing an elf executable

Hi Anastasios, On Thu, 2020-06-25 at 02:46 +0100, Anastasios Andronidis via Elfutils- devel wrote: > My end goal is to add a DT_NEEDED entry into an arbitrary elf file, > but before this I should just print the DT_NEEDED entries like this: > > ```C > > // Some code that copies a source_elf_file

Re: [PATCH] Complete PR25797 by modifying * debuginfod-client.c : check scheme instead of effective url so that user may abbreviate DEBUGINFOD_URL * tests/run-debuginfod-find.sh : add one test for sch

Hi Alice, On Mon, 2020-06-29 at 09:47 -0400, Alice Zhang via Elfutils-devel wrote: > Signed-off-by: Alice Zhang > --- > debuginfod/debuginfod-client.c | 17 - > tests/run-debuginfod-find.sh | 6 ++ > 2 files changed, 14 insertions(+), 9 deletions(-) The commit message is

[PATCH] libasm: Make libasm.h include work without relying on any other include.

/sourceware.org/bugzilla/show_bug.cgi?id=26176 Signed-off-by: Mark Wielaard --- libasm/ChangeLog | 4 libasm/libasm.h| 1 + tests/ChangeLog| 6 ++ tests/Makefile.am | 5 +++-- tests/run-test-includes.sh | 26 ++ 5 fi

[PATCH] libdwfl, src: Replace some asserts with proper check or error messages.

library code really shouldn't assert and for utilities a proper error message is better than crashing with an assert. https://sourceware.org/bugzilla/show_bug.cgi?id=26176 Signed-off-by: Mark Wielaard --- libdwfl/ChangeLog| 14 ++ libdwfl/argp-

Re: [PATCH] debuginfod: DEBUGINFOD_URLS should accept scheme-free urls

d-client.c (debuginfod_query_server): Use strncasecmp + to compare effective_url. Try CURLINFO_SCHEME as fallback. + 2020-06-19 Mark Wielaard * Makefile.am (bin_PROGRAMS): Guard with DEBUGINFOD and diff --git a/debuginfod/debuginfod-client.c b/debuginfod/debuginfod-client.c index c2e43f74..18

Re: [PATCH] libasm: Make libasm.h include work without relying on any other include.

On Sun, 2020-07-05 at 00:17 +0200, Mark Wielaard wrote: > The public headers should be usable when includes as is. > libasm.h wasn't because it was using gelf.h data structures without > include gelf.h. Include it now in libasm.h. > > Add a new testcase run-test-includes.s

[PATCH] configure.ac: Set -DBAD_FTS=1 also for CXXFLAGS.

debuginfod.cxx checks whether or not BAD_FTS is defined. This only matters on old 32bit arches. Signed-off-by: Mark Wielaard --- ChangeLog| 4 configure.ac | 3 ++- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index 99f50492..c6b526fe 100644 --- a

Re: [PATCH] libdwfl, src: Replace some asserts with proper check or error messages.

On Sun, Jul 05, 2020 at 12:33:38AM +0200, Mark Wielaard wrote: > library code really shouldn't assert and for utilities a proper > error message is better than crashing with an assert. Pushed to master.

[PATCH] elfclassify: Handle fwrite to stdout failing.

In theory an fwrite can fail, if so, signal it by calling issue (). Signed-off-by: Mark Wielaard --- src/ChangeLog | 4 src/elfclassify.c | 3 ++- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/src/ChangeLog b/src/ChangeLog index 0129b8bf..fa7411fd 100644 --- a/src

[PATCH] Only typedef Ebl once.

. This only affects internal code, since libebl.h isn't a public header. For internal code, only add the typedef in libebl.h when libasm.h hasn't been included. Make sure all code that includes both headers includes libasm.h first. Signed-off-by: Mark Wielaard --- libasm/ChangeLog | 4 ++

Re: [PATCH] configure.ac: Set -DBAD_FTS=1 also for CXXFLAGS.

On Sat, 2020-07-18 at 01:56 +0200, Mark Wielaard wrote: > debuginfod.cxx checks whether or not BAD_FTS is defined. > This only matters on old 32bit arches. Pushed to master.

Re: [PATCH] elfclassify: Handle fwrite to stdout failing.

On Sun, 2020-07-19 at 19:41 +0200, Mark Wielaard wrote: > In theory an fwrite can fail, if so, signal it by calling issue (). Pushed to master.

Re: [PATCH] Only typedef Ebl once.

On Sun, 2020-07-19 at 22:03 +0200, Mark Wielaard wrote: > Since commit 287a18452 libasm.h defines an opague Ebl handle. > This is fine, except for (internal) code that also includes libebl.h. > Since C11 having multiple typedefs for the same thing is fine, but we > do build using G

[PATCH] libelf: Fixup SHF_COMPRESSED sh_addralign in elf_update if necessary.

same for the sh_addralign field of a SHF_COMPRESSED section. Signed-off-by: Mark Wielaard --- libelf/ChangeLog | 5 + libelf/elf32_updatenull.c | 12 2 files changed, 17 insertions(+) diff --git a/libelf/ChangeLog b/libelf/ChangeLog index 8f6d2d2d..77044c1c 100644 --- a/libe

Re: [PATCH] libelf: Fixup SHF_COMPRESSED sh_addralign in elf_update if necessary.

On Wed, Aug 19, 2020 at 11:51:15PM +0200, Mark Wielaard wrote: > In elf_getdata.c we have the following to compensate for possibly > bad sh_addralign values of compressed sections: > > /* Compressed data has a header, but then compressed data. > Make sure to set t

[PATCH] readelf: It is not an error if there are no line number statements

It can happen that there are no line number statements at the end of a debug line section. So don't check that there are any more bytes after the last file entry. And print "No line number statements." libdw already got this corner case correct. Signed-off-by: Mark Wielaard ---

[PATCH 1/2] libelf: Sync elf.h from glibc.

Gets us DT_AARCH64_BTI_PLT and DT_AARCH64_PAC_PLT. Signed-off-by: Mark Wielaard --- libelf/ChangeLog | 4 libelf/elf.h | 2 ++ 2 files changed, 6 insertions(+) diff --git a/libelf/ChangeLog b/libelf/ChangeLog index 77044c1c..a3f15883 100644 --- a/libelf/ChangeLog +++ b/libelf

[PATCH 2/2] backends: Implement aarch64_dynamic_tag_name and aarch64_dynamic_tag_check

Signed-off-by: Mark Wielaard --- backends/ChangeLog| 7 +++ backends/aarch64_init.c | 2 ++ backends/aarch64_symbol.c | 26 ++ 3 files changed, 35 insertions(+) diff --git a/backends/ChangeLog b/backends/ChangeLog index c85dfd2a..ce4c971e 100644 --- a

Re: [PATCH] libdwfl: do not dlopen libdebuginfod.so in --disable-libdebuginfod mode

Hi Dmitry, On Thu, Aug 20, 2020 at 11:27:24PM +0300, Dmitry V. Levin wrote: > debuginfod-client.c used to try to dlopen libdebuginfod.so even if > libdebuginfod was completely disabled using --disable-libdebuginfod. > > Fix this by disabling build of debuginfod-client.c and disabling all > __libd

Re: [PATCH] readelf: It is not an error if there are no line number statements

On Wed, Aug 26, 2020 at 02:30:42PM +0200, Mark Wielaard wrote: > It can happen that there are no line number statements at the end of > a debug line section. So don't check that there are any more bytes > after the last file entry. And print "No line number statements." Pushed.

Re: [PATCH] libdwfl: do not dlopen libdebuginfod.so in --disable-libdebuginfod mode

Hi Dmitry, On Sun, Aug 30, 2020 at 11:49:10PM +0300, Dmitry V. Levin wrote: > On Sun, Aug 30, 2020 at 10:40:29PM +0200, Mark Wielaard wrote: > > To double check I removed all references to debuginfod_client from > > libdwflP.h when libdebuginfod is disabled. And things look fine i

[PATCH] readelf: base address entry can be first

In both debug_ranges and debug_loc a base address entry can be the first in the range or loc list. If so print the offset and set first to false. Otherwise don't print the entry list offset, but do set first to false. Signed-off-by: Mark Wielaard --- src/ChangeLog | 6 ++ src/read

Doing a 0.181 release (arm64 bti/pac issues)

Hi, On irc Martin was asking about doing a new release. I do think it is time for 0.181. But I am struggling a bit with Fedora arm64 which has turned on BTI/PAC which caused some issues: https://bugzilla.redhat.com/show_bug.cgi?id=1861423 I have patches for the dynamic tag parsing already posted

[PATCH] libebl: Handle aarch64 bti, pac bits in gnu property note

: Mark Wielaard --- libebl/ChangeLog | 5 ++ libebl/eblobjnote.c | 67 +++ tests/ChangeLog | 5 ++ tests/run-readelf-n.sh | 31 + tests/testfile-gnu-property-note

[PATCH] libdw: Remove duplicate local wildcards from map file.

We only need one local: * entry to capture all private local symbols. Signed-off-by: Mark Wielaard --- libdw/ChangeLog | 8 libdw/libdw.map | 10 -- 2 files changed, 8 insertions(+), 10 deletions(-) diff --git a/libdw/ChangeLog b/libdw/ChangeLog index 796dbfb7..13683ef0

Re: [PATCH] libebl: Handle aarch64 bti, pac bits in gnu property note

On Thu, 2020-09-03 at 11:56 +0200, Mark Wielaard wrote: > When building with gcc -mbranch-protection= we might get a gnu property > note indicating BTI (Branch Target Identification) and/or PAC (Pointer > Authentication Code) is being used. > > Add a small testcase to show eu

[PATCH] libdw,readelf: Recognize DW_CFA_AARCH64_negate_ra_state

es to show the value is correctly recognized. Also don't warn we cannot find any DWARF if we are just dumping frames (those will come from .eh_frame if there is no .debug_frame). Signed-off-by: Mark Wielaard --- libdw/ChangeLog | 12 +++ libdw/cfi.c | 41 +--

Re: [PATCH] libdw: Remove duplicate local wildcards from map file.

On Thu, Sep 03, 2020 at 07:57:15PM +0300, Dmitry V. Levin wrote: > On Thu, Sep 03, 2020 at 03:00:36PM +0200, Mark Wielaard wrote: > > We only need one local: * entry to capture all private local symbols. > > Yes, the first one (inside ELFUTILS_0.122) is enough. Thanks for the re

[COMMITTED] elflint: Handle .debug_line_str as .debug_str

The new DWARF5 .debug_line_str is identical to the .debug_str section. Signed-off-by: Mark Wielaard --- src/ChangeLog | 4 src/elflint.c | 1 + 2 files changed, 5 insertions(+) diff --git a/src/ChangeLog b/src/ChangeLog index 8eb2103d..e37f3587 100644 --- a/src/ChangeLog +++ b/src

Re: [PATCH 1/2] libelf: Sync elf.h from glibc.

On Sat, 2020-08-29 at 00:42 +0200, Mark Wielaard wrote: > Gets us DT_AARCH64_BTI_PLT and DT_AARCH64_PAC_PLT. Pushed.

Re: [PATCH 2/2] backends: Implement aarch64_dynamic_tag_name and aarch64_dynamic_tag_check

On Sat, 2020-08-29 at 00:42 +0200, Mark Wielaard wrote: > +2020-08-28 Mark Wielard > + > + * aarch64_init.c (aarch64_init): Hook dynamic_tag_name and > + dynamic_tag_check. > + * aarch64_symbol.c (aarch64_dynamic_tag_name): New function. > + (aarch64_dynamic_ta

Re: [PATCH] readelf: base address entry can be first

On Tue, 2020-09-01 at 23:48 +0200, Mark Wielaard wrote: > In both debug_ranges and debug_loc a base address entry can be the > first in the range or loc list. If so print the offset and set first > to false. Otherwise don't print the entry list offset, but do set > first to false. Pushed.

Re: [PATCH] libebl: Handle aarch64 bti, pac bits in gnu property note

On Thu, 2020-09-03 at 17:28 +0200, Mark Wielaard wrote: > On Thu, 2020-09-03 at 11:56 +0200, Mark Wielaard wrote: > > When building with gcc -mbranch-protection= we might get a gnu property > > note indicating BTI (Branch Target Identification) and/or PAC (Pointer > > Authenti

Re: [PATCH] libdw,readelf: Recognize DW_CFA_AARCH64_negate_ra_state

On Thu, 2020-09-03 at 17:58 +0200, Mark Wielaard wrote: > DW_CFA_AARCH64_negate_ra_state is used on aarch64 to indicate whether > or not the return address is mangled or not. This has the same value > as the DW_CFA_GNU_window_save. So we have to pass around the e_machine > value of th

Re: Doing a 0.181 release (arm64 bti/pac issues)

Hi, On Wed, 2020-09-02 at 15:03 +0200, Mark Wielaard wrote: > On irc Martin was asking about doing a new release. I do think it is > time for 0.181. But I am struggling a bit with Fedora arm64 which has > turned on BTI/PAC which caused some issues: > https://bugzilla.redhat.com/sho

[COMMITTED] Prepare for 0.181

Set version to 0.181. Update NEWS and elfutils.spec.in. Regenerate po/*.po files. Signed-off-by: Mark Wielaard --- ChangeLog | 5 + NEWS| 17 + config/ChangeLog| 4 + config/elfutils.spec.in | 13 + configure.ac| 2 +- po/ChangeLog

[PATCH] debuginfod: Fix BUILD_STATIC build.

The BUILD_STATIC build, as used by the coverage setup, was broken for debuginfod because the DUMMY_LIBDEBUGINFOD case was mixed up. It should include libcurl_LIBS when NOT doing a dummy build. Signed-off-by: Mark Wielaard --- debuginfod/ChangeLog | 5 + debuginfod/Makefile.am | 4 ++-- 2

elfutils 0.181 released

Alice Zhang (1): debuginfod: DEBUGINFOD_URLS should accept scheme-free urls Dmitry V. Levin (1): libdwfl: do not dlopen libdebuginfod.so in --disable-libdebuginfod mode Frank Ch. Eigler (1): PR26195: adapt debuginfod to API change in libmicrohttpd-0.9.71 Mark Wielaard (30): tests: Add

[COMMITTED] debuginfod-find.c: Fix license block comment. File is GPLv3+ only.

The license itself was correct, just the comment to get a copy of the LGPL was wrong. Signed-off-by: Mark Wielaard --- debuginfod/ChangeLog | 4 debuginfod/debuginfod-find.c | 6 +++--- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/debuginfod/ChangeLog b/debuginfod

Re: [PATCH] debuginfod: Fix BUILD_STATIC build.

On Tue, Sep 08, 2020 at 02:14:51PM +0200, Mark Wielaard wrote: > The BUILD_STATIC build, as used by the coverage setup, was broken for > debuginfod because the DUMMY_LIBDEBUGINFOD case was mixed up. It should > include libcurl_LIBS when NOT doing a dummy build. Pushed.

[PATCH] libdwfl: Add ZSTD support.

USE_INFLATE code path. But I felt that the many macros didn't really help understand the code. So the unzip routine has a slightly different code path for ZSTD. Signed-off-by: Mark Wielaard --- ChangeLog| 4 ++ config/ChangeLog | 5 ++ c

Re: PATCH: debuginfod, db size reduction

Hi Frank, On Fri, 2020-09-18 at 13:41 -0400, Frank Ch. Eigler via Elfutils-devel wrote: > Since PR25548, we let debuginfod answer /buildid/HEX/source/PATH > queries with both canonicalized and raw PATHs. It canonicalizes > incoming paths, but still stored the raw paths in the database too. > This

Re: [PATCH] libdwfl: Add ZSTD support.

Hi, On Fri, 2020-09-18 at 15:17 +0200, Mark Wielaard wrote: > Newer kernels might be compressed using ZSTD add support to libdwfl > open so we can can automatically read ZSTD compressed files and > kernel images. > > The support is very similar to the bzip2 and lzma support,

[PATCH] readelf: Support showing view pairs in loclists as GNU extension.

View pairs are encoded in .debug_loclists as they are encoded in .debug_locs sections by the GCC compiler. Scan for DW_AT_GNU_locviews attributes that point to the view pairs just in front of the actual location lists. Signed-off-by: Mark Wielaard --- src/ChangeLog | 9 ++ src/readelf.c

Re: [PATCH] readelf: Support showing view pairs in loclists as GNU extension.

On Mon, Sep 28, 2020 at 09:43:44AM +0200, Mark Wielaard wrote: > View pairs are encoded in .debug_loclists as they are encoded in > .debug_locs sections by the GCC compiler. Scan for DW_AT_GNU_locviews > attributes that point to the view pairs just in front of the actual > location lists. Pushed.

Re: [PATCH] Add a --{enable,disable}-progs configure option

Hi, On Thu, Oct 01, 2020 at 08:34:47PM +0200, Fabrice Fontaine via Elfutils-devel wrote: > From: Thomas Petazzoni > > Add a --{enable,disable}-progs configuration option to elfutils. This > allows to selectively disable the compilation of the elfutils programs > (in which case only the librarie

Re: [patch] PR25461 default DEBUGINFOD_URLS

Hi Frank, On Thu, Oct 01, 2020 at 05:03:52PM -0400, Frank Ch. Eigler via Elfutils-devel wrote: > To give distros a simple knob to let their userbase enjoy debuginfod > services, we add a --enable-debuginfod-urls[=URLS] to the configury. > This propagates to $DEBUGINFOD_URLS environment variable s

<    13   14   15   16   17   18   19   20   21   22   >