]> git.ipfire.org Git - thirdparty/elfutils.git/commitdiff
Fix dwfl_report_elf BASE alignment
authorJan Kratochvil <jan.kratochvil@redhat.com>
Tue, 19 Mar 2013 12:40:13 +0000 (13:40 +0100)
committerJan Kratochvil <jan.kratochvil@redhat.com>
Wed, 20 Mar 2013 15:25:08 +0000 (16:25 +0100)
libdwfl/
* dwfl_report_elf.c (__libdwfl_report_elf): Remove BASE aligning.

tests/
* Makefile.am (check_PROGRAMS): Add dwfl-report-elf-align.
(TESTS): Add run-dwfl-report-elf-align.sh.
(EXTRA_DIST): Add run-dwfl-report-elf-align.sh and
testfile-dwfl-report-elf-align-shlib.so.bz2 .
(dwfl_report_elf_align_LDADD): New.
* dwfl-report-elf-align.c: New file.
* run-dwfl-report-elf-align.sh: New file.
* testfile-dwfl-report-elf-align-shlib.so.bz2: New file.

Signed-off-by: Jan Kratochvil <jan.kratochvil@redhat.com>
libdwfl/ChangeLog
libdwfl/dwfl_report_elf.c
tests/ChangeLog
tests/Makefile.am
tests/dwfl-report-elf-align.c [new file with mode: 0644]
tests/run-dwfl-report-elf-align.sh [new file with mode: 0755]
tests/testfile-dwfl-report-elf-align-shlib.so.bz2 [new file with mode: 0755]

index 15a6c9c9e4a1fcd6ab32d3a1286c0eade66efede..41fc84bb9b29b6ba5b1cac7cc683b886e02e0bb9 100644 (file)
@@ -1,3 +1,7 @@
+2013-03-20  Jan Kratochvil  <jan.kratochvil@redhat.com>
+
+       * dwfl_report_elf.c (__libdwfl_report_elf): Remove BASE aligning.
+
 2013-03-12  Mark Wielaard  <mjw@redhat.com>
 
        * dwfl_getsrclines.c (dwfl_getsrclines): Return 0 on success.
index 174ce7ce3e0387d1a72a8a78c7d33a794746f6a7..d7061704cbc57f780712a251ec14df1323ccad1e 100644 (file)
@@ -181,8 +181,6 @@ __libdwfl_report_elf (Dwfl *dwfl, const char *name, const char *file_name,
            {
              vaddr = ph->p_vaddr & -ph->p_align;
              address_sync = ph->p_vaddr + ph->p_memsz;
-             if ((base & (ph->p_align - 1)) != 0)
-               base = (base + ph->p_align - 1) & -ph->p_align;
              start = base + vaddr;
              break;
            }
index ca06a55c22ab33732031c9c7bb28f124b436d5eb..62b589dc86241868e8df054ff13d1fad9e1aa13d 100644 (file)
@@ -1,3 +1,14 @@
+2013-03-20  Jan Kratochvil  <jan.kratochvil@redhat.com>
+
+       * Makefile.am (check_PROGRAMS): Add dwfl-report-elf-align.
+       (TESTS): Add run-dwfl-report-elf-align.sh.
+       (EXTRA_DIST): Add run-dwfl-report-elf-align.sh and
+       testfile-dwfl-report-elf-align-shlib.so.bz2 .
+       (dwfl_report_elf_align_LDADD): New.
+       * dwfl-report-elf-align.c: New file.
+       * run-dwfl-report-elf-align.sh: New file.
+       * testfile-dwfl-report-elf-align-shlib.so.bz2: New file.
+
 2013-03-12  Mark Wielaard  <mjw@redhat.com>
 
        * run-dwfllines.sh: New test.
index a5f7a8c0d538f68f31349ce8b9557a56742f9bed..9b0bbf1a7f7b7bae47d3ad9a3498c11d42c0eebd 100644 (file)
@@ -51,7 +51,8 @@ check_PROGRAMS = arextract arsymtest newfile saridx scnnames sectiondump \
                  dwfl-bug-getmodules dwarf-getmacros addrcfi \
                  test-flag-nobits dwarf-getstring rerequest_tag \
                  alldts md5-sha1-test typeiter low_high_pc \
-                 test-elf_cntl_gelf_getshdr dwflsyms dwfllines
+                 test-elf_cntl_gelf_getshdr dwflsyms dwfllines \
+                 dwfl-report-elf-align
 asm_TESTS = asm-tst1 asm-tst2 asm-tst3 asm-tst4 asm-tst5 \
            asm-tst6 asm-tst7 asm-tst8 asm-tst9
 
@@ -85,7 +86,8 @@ TESTS = run-arextract.sh run-arsymtest.sh newfile test-nlist \
        run-readelf-d.sh run-readelf-gdb_index.sh run-unstrip-n.sh \
        run-low_high_pc.sh run-macro-test.sh run-elf_cntl_gelf_getshdr.sh \
        run-test-archive64.sh run-readelf-vmcoreinfo.sh \
-       run-readelf-mixed-corenote.sh run-dwfllines.sh
+       run-readelf-mixed-corenote.sh run-dwfllines.sh \
+       run-dwfl-report-elf-align.sh
 
 if !STANDALONE
 check_PROGRAMS += msg_tst md5-sha1-test
@@ -193,7 +195,8 @@ EXTRA_DIST = run-arextract.sh run-arsymtest.sh \
             testfile65.bz2 testfile67.bz2 testfile68.bz2 \
             testfile69.core.bz2 testfile69.so.bz2 \
             testfile70.core.bz2 testfile70.exec.bz2 \
-            run-dwfllines.sh
+            run-dwfllines.sh run-dwfl-report-elf-align.sh \
+            testfile-dwfl-report-elf-align-shlib.so.bz2
 
 if USE_VALGRIND
 valgrind_cmd="valgrind -q --trace-children=yes --error-exitcode=1 --run-libc-freeres=no"
@@ -302,6 +305,7 @@ low_high_pc_LDADD = $(libdw) $(libelf) $(libmudflap)
 test_elf_cntl_gelf_getshdr_LDADD = $(libelf) $(libmudflap)
 dwflsyms_LDADD = $(libdw) $(libelf) $(libmudflap)
 dwfllines_LDADD = $(libdw) $(libelf) $(libmudflap)
+dwfl_report_elf_align_LDADD = $(libdw) $(libmudflap)
 
 if GCOV
 check: check-am coverage
diff --git a/tests/dwfl-report-elf-align.c b/tests/dwfl-report-elf-align.c
new file mode 100644 (file)
index 0000000..0e8bfe3
--- /dev/null
@@ -0,0 +1,72 @@
+/* Test program for dwfl_report_elf incorrect BASE alignment.
+   Copyright (C) 2013 Red Hat, Inc.
+   This file is part of elfutils.
+
+   This file is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 3 of the License, or
+   (at your option) any later version.
+
+   elfutils is distributed in the hope that it will be useful, but
+   WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
+
+   You should have received a copy of the GNU General Public License
+   along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
+
+#include <config.h>
+#include <assert.h>
+#include <inttypes.h>
+#include <stdio.h>
+#include <stdio_ext.h>
+#include <error.h>
+#include <locale.h>
+#include <string.h>
+#include <stdlib.h>
+#include ELFUTILS_HEADER(dwfl)
+
+
+static const Dwfl_Callbacks offline_callbacks =
+  {
+    .find_debuginfo = INTUSE(dwfl_standard_find_debuginfo),
+    .section_address = INTUSE(dwfl_offline_section_address),
+  };
+
+
+int
+main (int argc, char **argv)
+{
+  /* We use no threads here which can interfere with handling a stream.  */
+  (void) __fsetlocking (stdout, FSETLOCKING_BYCALLER);
+
+  /* Set locale.  */
+  (void) setlocale (LC_ALL, "");
+
+  if (argc != 5)
+    error (1, 0, "dwfl-report-elf-align shlib.so base funcaddr funcname");
+    
+  Dwfl *dwfl = dwfl_begin (&offline_callbacks);
+  assert (dwfl != NULL);
+
+  char *endptr;
+  uintptr_t base = strtoull (argv[2], &endptr, 0);
+  assert (endptr && !*endptr);
+
+  Dwfl_Module *mod = dwfl_report_elf (dwfl, argv[1], argv[1], -1, base);
+  assert (mod != NULL);
+
+  uintptr_t funcaddr = strtoull (argv[3], &endptr, 0);
+  assert (endptr && !*endptr);
+
+  Dwfl_Module *mod_found = dwfl_addrmodule (dwfl, funcaddr);
+  assert (mod_found == mod);
+
+  const char *symname = dwfl_module_addrname (mod, funcaddr);
+  assert (symname != NULL);
+  assert (strcmp (symname, argv[4]) == 0);
+
+  dwfl_end (dwfl);
+
+  return 0;
+}
diff --git a/tests/run-dwfl-report-elf-align.sh b/tests/run-dwfl-report-elf-align.sh
new file mode 100755 (executable)
index 0000000..d032aba
--- /dev/null
@@ -0,0 +1,44 @@
+#! /bin/sh
+# Copyright (C) 2013 Red Hat, Inc.
+# This file is part of elfutils.
+#
+# This file is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 3 of the License, or
+# (at your option) any later version.
+#
+# elfutils is distributed in the hope that it will be useful, but
+# WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program.  If not, see <http://www.gnu.org/licenses/>.
+
+. $srcdir/test-subr.sh
+
+testfiles testfile-dwfl-report-elf-align-shlib.so
+
+# /proc/PID/maps when the process was running:
+# 7f3560c92000-7f3560c93000 r-xp 00000000 fd:02 25037063 testfile-dwfl-report-elf-align-shlib.so
+# 7f3560c93000-7f3560e92000 ---p 00001000 fd:02 25037063 testfile-dwfl-report-elf-align-shlib.so
+# 7f3560e92000-7f3560e93000 rw-p 00000000 fd:02 25037063 testfile-dwfl-report-elf-align-shlib.so
+# testfile-dwfl-report-elf-align-shlib.so:
+# Program Headers:
+#   Type           Offset   VirtAddr           PhysAddr           FileSiz  MemSiz   Flg Align
+#   LOAD           0x000000 0x0000000000000000 0x0000000000000000 0x00065c 0x00065c R E 0x200000
+#   LOAD           0x000660 0x0000000000200660 0x0000000000200660 0x0001f0 0x000200 RW  0x200000
+# Symbol table '.dynsym' contains 12 entries:
+#    Num:    Value          Size Type    Bind   Vis      Ndx Name
+#      8: 000000000000057c    11 FUNC    GLOBAL DEFAULT   11 shlib
+# GDB output showing proper relocation:
+# #1  0x00007f3560c92585 in shlib () from ./testfile-dwfl-report-elf-align-shlib.so
+#
+# 0x7f3560c92000 is VMA address of first byte of testfile-dwfl-report-elf-align-shlib.so.
+# 0x7f3560c92585 = 0x7f3560c92000 + 0x585
+# where 0x585 is any address inside the shlib function: 0x57c .. 0x57c + 11 -1
+
+testrun ./dwfl-report-elf-align ./testfile-dwfl-report-elf-align-shlib.so \
+                               0x7f3560c92000 0x7f3560c92585 shlib
+
+exit 0
diff --git a/tests/testfile-dwfl-report-elf-align-shlib.so.bz2 b/tests/testfile-dwfl-report-elf-align-shlib.so.bz2
new file mode 100755 (executable)
index 0000000..1f35df9
Binary files /dev/null and b/tests/testfile-dwfl-report-elf-align-shlib.so.bz2 differ