diff options
author | Renlin Li <renlin.li@arm.com> | 2018-02-03 13:18:17 +0000 |
---|---|---|
committer | Renlin Li <renlin.li@arm.com> | 2018-02-05 18:25:28 +0000 |
commit | b01452b1d44a586f4ecf5cf02ffc0643e4962324 (patch) | |
tree | 35b03895d1ff4c7b4661b8ef139e9b2fe00cf632 | |
parent | f4858ef234199d88ff0430ed476a5160ef1b38ab (diff) | |
download | binutils-gdb-b01452b1d44a586f4ecf5cf02ffc0643e4962324.tar.gz |
[PR22764][LD][AARCH64]Allow R_AARCH64_ABS16 and R_AARCH64_ABS32 against absolution symbol or undefine symbol in shared object.
backport from mainline
bfd/
2018-02-05 Renlin Li <renlin.li@arm.com>
PR ld/22764
* elfnn-aarch64.c (elfNN_aarch64_check_relocs): Relax the
R_AARCH64_ABS32 and R_AARCH64_ABS16 for absolute symbol. Apply the
check for writeable section as well.
ld/
2018-02-05 Renlin Li <renlin.li@arm.com>
PR ld/22764
* testsuite/ld-aarch64/emit-relocs-258.s: Define symbol as an address.
* testsuite/ld-aarch64/emit-relocs-259.s: Likewise.
* testsuite/ld-aarch64/aarch64-elf.exp: Run new test.
* testsuite/ld-aarch64/pr22764.s: New.
* testsuite/ld-aarch64/pr22764.d: New.
-rw-r--r-- | bfd/ChangeLog | 7 | ||||
-rw-r--r-- | bfd/elfnn-aarch64.c | 15 | ||||
-rw-r--r-- | ld/ChangeLog | 8 | ||||
-rw-r--r-- | ld/testsuite/ld-aarch64/aarch64-elf.exp | 1 | ||||
-rw-r--r-- | ld/testsuite/ld-aarch64/emit-relocs-258.s | 3 | ||||
-rw-r--r-- | ld/testsuite/ld-aarch64/emit-relocs-259.s | 3 | ||||
-rw-r--r-- | ld/testsuite/ld-aarch64/pr22764.d | 18 | ||||
-rw-r--r-- | ld/testsuite/ld-aarch64/pr22764.s | 6 |
8 files changed, 56 insertions, 5 deletions
diff --git a/bfd/ChangeLog b/bfd/ChangeLog index 92e26a8aa23..9fe447a29e5 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,10 @@ +2018-02-05 Renlin Li <renlin.li@arm.com> + + PR ld/22764 + * elfnn-aarch64.c (elfNN_aarch64_check_relocs): Relax the + R_AARCH64_ABS32 and R_AARCH64_ABS16 for absolute symbol. Apply the + check for writeable section as well. + 2018-01-27 Nick Clifton <nickc@redhat.com> Back to development. diff --git a/bfd/elfnn-aarch64.c b/bfd/elfnn-aarch64.c index d5711e0eb1d..973188220ba 100644 --- a/bfd/elfnn-aarch64.c +++ b/bfd/elfnn-aarch64.c @@ -7074,10 +7074,19 @@ elfNN_aarch64_check_relocs (bfd *abfd, struct bfd_link_info *info, #if ARCH_SIZE == 64 case BFD_RELOC_AARCH64_32: #endif - if (bfd_link_pic (info) - && (sec->flags & SEC_ALLOC) != 0 - && (sec->flags & SEC_READONLY) != 0) + if (bfd_link_pic (info) && (sec->flags & SEC_ALLOC) != 0) { + if (h != NULL + /* This is an absolute symbol. It represents a value instead + of an address. */ + && ((h->root.type == bfd_link_hash_defined + && bfd_is_abs_section (h->root.u.def.section)) + /* This is an undefined symbol. */ + || h->root.type == bfd_link_hash_undefined)) + break; + + /* For local symbols, defined global symbols in a non-ABS section, + it is assumed that the value is an address. */ int howto_index = bfd_r_type - BFD_RELOC_AARCH64_RELOC_START; _bfd_error_handler /* xgettext:c-format */ diff --git a/ld/ChangeLog b/ld/ChangeLog index ddc1d88f3c1..e871cef7c07 100644 --- a/ld/ChangeLog +++ b/ld/ChangeLog @@ -1,3 +1,11 @@ +2018-02-05 Renlin Li <renlin.li@arm.com> + + PR ld/22764 + * testsuite/ld-aarch64/emit-relocs-258.s: Define symbol as an address. + * testsuite/ld-aarch64/emit-relocs-259.s: Likewise. + * testsuite/ld-aarch64/pr22764.s: New. + * testsuite/ld-aarch64/pr22764.d: New. + 2018-02-05 Nick Clifton <nickc@redhat.com> * po/pt_BR.po: Updated Brazilian Portuguese translation. diff --git a/ld/testsuite/ld-aarch64/aarch64-elf.exp b/ld/testsuite/ld-aarch64/aarch64-elf.exp index f31089361b8..d766f3736bc 100644 --- a/ld/testsuite/ld-aarch64/aarch64-elf.exp +++ b/ld/testsuite/ld-aarch64/aarch64-elf.exp @@ -275,6 +275,7 @@ run_dump_test "pr17415" run_dump_test_lp64 "tprel_g2_overflow" run_dump_test "tprel_add_lo12_overflow" run_dump_test "protected-data" +run_dump_test_lp64 "pr22764" # ifunc tests run_dump_test "ifunc-1" diff --git a/ld/testsuite/ld-aarch64/emit-relocs-258.s b/ld/testsuite/ld-aarch64/emit-relocs-258.s index f7247762431..87bb657c5d9 100644 --- a/ld/testsuite/ld-aarch64/emit-relocs-258.s +++ b/ld/testsuite/ld-aarch64/emit-relocs-258.s @@ -1,5 +1,6 @@ +.global dummy .text - +dummy: ldr x0, .L1 .L1: diff --git a/ld/testsuite/ld-aarch64/emit-relocs-259.s b/ld/testsuite/ld-aarch64/emit-relocs-259.s index 7e1ba3ceb40..0977c9d869f 100644 --- a/ld/testsuite/ld-aarch64/emit-relocs-259.s +++ b/ld/testsuite/ld-aarch64/emit-relocs-259.s @@ -1,5 +1,6 @@ +.global dummy .text - +dummy: ldr x0, .L1 .L1: diff --git a/ld/testsuite/ld-aarch64/pr22764.d b/ld/testsuite/ld-aarch64/pr22764.d new file mode 100644 index 00000000000..997519f4699 --- /dev/null +++ b/ld/testsuite/ld-aarch64/pr22764.d @@ -0,0 +1,18 @@ +#source: pr22764.s +#ld: -shared -T relocs.ld -defsym sym_abs1=0x1 -defsym sym_abs2=0x2 -defsym sym_abs3=0x3 -e0 --emit-relocs +#notarget: aarch64_be-*-* +#objdump: -dr +#... + +Disassembly of section \.text: + +0000000000010000 \<\.text\>: + 10000: d503201f nop + ... + 10004: R_AARCH64_ABS64 sym_abs1 + 1000c: 00000002 \.word 0x00000002 + 1000c: R_AARCH64_ABS32 sym_abs2 + 10010: 0003 \.short 0x0003 + 10010: R_AARCH64_ABS16 sym_abs3 + 10012: 0000 \.short 0x0000 + 10014: d503201f nop diff --git a/ld/testsuite/ld-aarch64/pr22764.s b/ld/testsuite/ld-aarch64/pr22764.s new file mode 100644 index 00000000000..25e36b4a129 --- /dev/null +++ b/ld/testsuite/ld-aarch64/pr22764.s @@ -0,0 +1,6 @@ + .text + nop + .xword sym_abs1 + .word sym_abs2 + .short sym_abs3 + nop |