Re: [PATCH] strings: Produce error when section data falls outside file.

2014-12-26 Thread Alexander Cherepanov
On 2014-12-27 01:00, Mark Wielaard wrote: diff --git a/src/strings.c b/src/strings.c index f60e4b4..d1eb7b2 100644 --- a/src/strings.c +++ b/src/strings.c @@ -725,8 +725,21 @@ read_elf (Elf *elf, int fd, const char *fname, off64_t fdlen) actually have content. */ if (shdr != NU

[PATCH] strings: Produce error when section data falls outside file.

2014-12-26 Thread Mark Wielaard
https://bugzilla.redhat.com/show_bug.cgi?id=1170810 Reported-by: Alexander Cherepanov Signed-off-by: Mark Wielaard --- src/ChangeLog | 5 + src/strings.c | 17 +++-- 2 files changed, 20 insertions(+), 2 deletions(-) diff --git a/src/ChangeLog b/src/ChangeLog index 904b3c9..66d

Re: [PATCH] libdw: Search for the last matching address with dwarf_getsrc_die.

2014-12-26 Thread Josh Stone
On 12/24/2014 05:27 AM, Mark Wielaard wrote: > In commit 7d9b5a dwfl_module_getsrc was changed so that it returns the last > line record <= addr, rather than returning immediately on a match. This > changes dwarf_getsrc_die to do the same. And it adds a new test that checks > this by comparing agai

[PATCH] nm: Guard against divide by zero in error check.

2014-12-26 Thread Mark Wielaard
Also add section index number in error message. https://bugzilla.redhat.com/show_bug.cgi?id=1170810 Reported-by: Alexander Cherepanov Signed-off-by: Mark Wielaard --- src/ChangeLog | 5 + src/nm.c | 13 - 2 files changed, 13 insertions(+), 5 deletions(-) diff --git a/src

[PATCH] nm: Handle/Skip /SYM64/ index entries.

2014-12-26 Thread Mark Wielaard
Add a testcase to show eu-nm now handles archives with 64-bit symbol table. Signed-off-by: Mark Wielaard --- src/ChangeLog | 4 src/nm.c| 3 ++- tests/ChangeLog | 4 tests/run-test-archive64.sh | 14 +- 4 files changed, 23 in

[PATCH] nm: Stop processing ar members on first invalid offset.

2014-12-26 Thread Mark Wielaard
Otherwise we will keep looping on that same invalid entry. https://bugzilla.redhat.com/show_bug.cgi?id=1170810 Reported-by: Alexander Cherepanov Signed-off-by: Mark Wielaard --- src/ChangeLog | 4 src/nm.c | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/src/Chan