diff options
author | Maciej W. Rozycki <macro@mips.com> | 2018-04-09 13:42:01 +0100 |
---|---|---|
committer | Maciej W. Rozycki <macro@mips.com> | 2018-04-09 13:42:01 +0100 |
commit | d52e3d06e5db7041913278caace3389504c0478c (patch) | |
tree | 1480a5d003446c1f36261e9439f9291adb23647a | |
parent | aec2e0d252342aa7575aff608e999a36f27a9147 (diff) | |
download | gdb-d52e3d06e5db7041913278caace3389504c0478c.zip gdb-d52e3d06e5db7041913278caace3389504c0478c.tar.gz gdb-d52e3d06e5db7041913278caace3389504c0478c.tar.bz2 |
binutils/testsuite: Fix a crash with STN_UNDEF in relocation
Verify that `strip' completes successfully and a correct relocation
entry is copied for a relocation encountered with the STN_UNDEF symbol
index.
binutils/
* testsuite/binutils-all/strip-15.d: New test.
* testsuite/binutils-all/strip-15rel.s: New test source.
* testsuite/binutils-all/strip-15rela.s: New test source.
* testsuite/binutils-all/strip-15mips64.s: New test source.
* testsuite/binutils-all/objcopy.exp: Run the new test.
-rw-r--r-- | binutils/ChangeLog | 8 | ||||
-rw-r--r-- | binutils/testsuite/binutils-all/objcopy.exp | 5 | ||||
-rw-r--r-- | binutils/testsuite/binutils-all/strip-15.d | 8 | ||||
-rw-r--r-- | binutils/testsuite/binutils-all/strip-15mips64.s | 14 | ||||
-rw-r--r-- | binutils/testsuite/binutils-all/strip-15rel.s | 23 | ||||
-rw-r--r-- | binutils/testsuite/binutils-all/strip-15rela.s | 27 |
6 files changed, 85 insertions, 0 deletions
diff --git a/binutils/ChangeLog b/binutils/ChangeLog index 0c1b95d..c7a631a 100644 --- a/binutils/ChangeLog +++ b/binutils/ChangeLog @@ -1,5 +1,13 @@ 2018-04-09 Maciej W. Rozycki <macro@mips.com> + * testsuite/binutils-all/strip-15.d: New test. + * testsuite/binutils-all/strip-15rel.s: New test source. + * testsuite/binutils-all/strip-15rela.s: New test source. + * testsuite/binutils-all/strip-15mips64.s: New test source. + * testsuite/binutils-all/objcopy.exp: Run the new test. + +2018-04-09 Maciej W. Rozycki <macro@mips.com> + * testsuite/binutils-all/strip-14.d: New test. * testsuite/binutils-all/strip-14rel.s: New test source. * testsuite/binutils-all/strip-14rela.s: New test source. diff --git a/binutils/testsuite/binutils-all/objcopy.exp b/binutils/testsuite/binutils-all/objcopy.exp index 532bdc6..366a4a3 100644 --- a/binutils/testsuite/binutils-all/objcopy.exp +++ b/binutils/testsuite/binutils-all/objcopy.exp @@ -1123,6 +1123,11 @@ if [is_elf_format] { [list as "--defsym RELOC=${reloc}"] \ [list as [expr {[is_elf64 tmpdir/bintest.o] \ ? "--defsym ELF64=1" : ""}]]] + run_dump_test "strip-15" [list \ + [list source strip-15${reloc_format}.s] \ + [list as "--defsym RELOC=${reloc}"] \ + [list as [expr {[is_elf64 tmpdir/bintest.o] \ + ? "--defsym ELF64=1" : ""}]]] # This requires STB_GNU_UNIQUE support with OSABI set to GNU. if { [supports_gnu_unique] } { diff --git a/binutils/testsuite/binutils-all/strip-15.d b/binutils/testsuite/binutils-all/strip-15.d new file mode 100644 index 0000000..03d7d1a --- /dev/null +++ b/binutils/testsuite/binutils-all/strip-15.d @@ -0,0 +1,8 @@ +#PROG: strip +#strip: -g +#readelf: -r + +Relocation section '\.rela?\.text' at offset .* contains 2 entries: + *Offset * Info * Type * Sym\. *Value * Sym\. *Name(?: * \+ * Addend)? +0+00 * 0+0(?:1|32|103) * R_[^ ]* *(?: * 55aa)? +#pass diff --git a/binutils/testsuite/binutils-all/strip-15mips64.s b/binutils/testsuite/binutils-all/strip-15mips64.s new file mode 100644 index 0000000..a15c377 --- /dev/null +++ b/binutils/testsuite/binutils-all/strip-15mips64.s @@ -0,0 +1,14 @@ + .text +foo: + .dc.l 0x12345678 + + .section .rela.text + .dc.a 0 + .dc.l 0x00000000 + .dc.b 0, 0, 0, RELOC + .dc.a 0x000055aa + + .dc.a 0 + .dc.l 0 + .dc.b 0, 0, 0, 0 + .dc.a 0 diff --git a/binutils/testsuite/binutils-all/strip-15rel.s b/binutils/testsuite/binutils-all/strip-15rel.s new file mode 100644 index 0000000..1819a53 --- /dev/null +++ b/binutils/testsuite/binutils-all/strip-15rel.s @@ -0,0 +1,23 @@ + .text +foo: + .dc.l 0x12345678 + + .section .rel.text + .ifdef ELF64 + + .dc.a 0 + .dc.a RELOC + + .dc.a 0 + .dc.a 0 + .else + + # Some targets, such as `m68hc11-*', use 16-bit addresses. + # With them `.dc.a' emits 16-bit quantities, so we need to use + # `.dc.l' for 32-bit relocation data. + .dc.l 0 + .dc.l RELOC + + .dc.l 0 + .dc.l 0 + .endif diff --git a/binutils/testsuite/binutils-all/strip-15rela.s b/binutils/testsuite/binutils-all/strip-15rela.s new file mode 100644 index 0000000..7bcd85e --- /dev/null +++ b/binutils/testsuite/binutils-all/strip-15rela.s @@ -0,0 +1,27 @@ + .text +foo: + .dc.l 0x12345678 + + .section .rela.text + .ifdef ELF64 + + .dc.a 0 + .dc.a RELOC + .dc.a 0x00000000000055aa + + .dc.a 0 + .dc.a 0 + .dc.a 0 + .else + + # Some targets, such as `h8300-*' or `ip2k-*', use 16-bit addresses. + # With them `.dc.a' emits 16-bit quantities, so we need to use + # `.dc.l' for 32-bit relocation data. + .dc.l 0 + .dc.l RELOC + .dc.l 0x000055aa + + .dc.l 0 + .dc.l 0 + .dc.l 0 + .endif |