[Bug libdw/29434] Memory leak in `dwarf_getscopes`

2022-11-03 Thread pablogsal at gmail dot com via Elfutils-devel
https://sourceware.org/bugzilla/show_bug.cgi?id=29434 --- Comment #15 from Pablo Galindo Salgado --- Here is the parts that reference the parts: __libdw_visit_scopes for pc 0xf5b23, cudie [29ce89] __libdw_visit_scopes result 0, scopes (nil), inlined 0, nscopes 0 __libdw_visit_scopes for pc 0x252

[Bug libdw/29434] Memory leak in `dwarf_getscopes`

2022-11-03 Thread pablogsal at gmail dot com via Elfutils-devel
https://sourceware.org/bugzilla/show_bug.cgi?id=29434 --- Comment #14 from Pablo Galindo Salgado --- I can tell you that the file finishes with: pc_record => [5d9f] pc_record <= [5d9f] 3 __libdw_visit_scopes result 3, scopes 0xa689d0, inlined 0, nscopes 3 :) -- You are receiving this mail

[Bug libdw/29434] Memory leak in `dwarf_getscopes`

2022-11-03 Thread pablogsal at gmail dot com via Elfutils-devel
https://sourceware.org/bugzilla/show_bug.cgi?id=29434 --- Comment #13 from Pablo Galindo Salgado --- Hum, it seems that I cannot attach the output because is 87 MB of output. -- You are receiving this mail because: You are on the CC list for the bug.

[Bug libdw/29434] Memory leak in `dwarf_getscopes`

2022-11-03 Thread pablogsal at gmail dot com via Elfutils-devel
https://sourceware.org/bugzilla/show_bug.cgi?id=29434 --- Comment #12 from Pablo Galindo Salgado --- Hopefully I can report back with the output in a few days. Apologies for the delay! -- You are receiving this mail because: You are on the CC list for the bug.

[Bug libdw/29434] Memory leak in `dwarf_getscopes`

2022-11-03 Thread pablogsal at gmail dot com via Elfutils-devel
https://sourceware.org/bugzilla/show_bug.cgi?id=29434 --- Comment #11 from Pablo Galindo Salgado --- Unfortunately I cannot easily patch dwarf_getscopes and run the experiment :( -- You are receiving this mail because: You are on the CC list for the bug.

☺ Buildbot (GNU Toolchain): elfutils - build successful (master)

2022-11-03 Thread builder--- via Elfutils-devel
A restored build has been detected on builder elfutils-fedora-x86_64 while building elfutils. Full details are available at: https://builder.sourceware.org/buildbot/#builders/59/builds/108 Build state: build successful Revision: 52a6a3110e019d696284fdd822c2a2f0987dded2 Worker: bb2-2 Build Re

☠ Buildbot (GNU Toolchain): elfutils - failed test (failure) (master)

2022-11-03 Thread builder--- via Elfutils-devel
A new failure has been detected on builder elfutils-fedora-x86_64 while building elfutils. Full details are available at: https://builder.sourceware.org/buildbot/#builders/59/builds/107 Build state: failed test (failure) Revision: b0a0235771906e3bcd6174c4e3c020b5522b0be5 Worker: bb1-2 Build

☺ Buildbot (GNU Toolchain): elfutils - build successful (master)

2022-11-03 Thread builder--- via Elfutils-devel
A restored build has been detected on builder elfutils-fedora-x86_64 while building elfutils. Full details are available at: https://builder.sourceware.org/buildbot/#builders/59/builds/106 Build state: build successful Revision: 75f2de448f311807e2493f2a37a980e2d872b229 Worker: bbo1-1 Build R

patch, debuginfod

2022-11-03 Thread Frank Ch. Eigler via Elfutils-devel
Hi - Showing diff -w to omit reindentation whitespace noise. Maybe worth backporting to 0.188 releases, could affect federation intermediate servers with burst workload. commit ec166cf3c8d825a2f02aca448a0823de12e78991 (HEAD -> master) Author: Frank Ch. Eigler Date: Thu Nov 3 10:07:31 2022 -0

[COMMITTED] readelf: Check gelf_getdyn doesn't return NULL

2022-11-03 Thread Mark Wielaard
Signed-off-by: Mark Wielaard --- src/ChangeLog | 5 + src/readelf.c | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/src/ChangeLog b/src/ChangeLog index 0c5ab37e..66428b70 100644 --- a/src/ChangeLog +++ b/src/ChangeLog @@ -1,3 +1,8 @@ +2022-11-03 Mark Wielaard + +

[COMMITTED] libdw: Don't dereference and assign values we are skipping

2022-11-03 Thread Mark Wielaard
We don't use the FDE address encoding byte, so no reason to read and store it. Just skip past it. Signed-off-by: Mark Wielaard --- libdw/ChangeLog| 5 + libdw/dwarf_next_cfi.c | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/libdw/ChangeLog b/libdw/ChangeLog inde

[COMMITTED] readelf: Check phdr != NULL or shdr != NULL in handle_dynamic.

2022-11-03 Thread Mark Wielaard
The compiler doesn't know that when use_dynamic_segment is true, then phdr should/will be non-NULL and otherwise shdr is non-NULL. Add explicit checks to help the compiler out and in case an error is made calling the handle_dynamic function. Signed-off-by: Mark Wielaard --- src/ChangeLog | 5 ++