diff options
author | H.J. Lu <hjl.tools@gmail.com> | 2015-12-01 14:45:51 -0800 |
---|---|---|
committer | H.J. Lu <hjl.tools@gmail.com> | 2015-12-01 14:45:51 -0800 |
commit | ead3d5427a2df5e33316d4ad045510c1d2078c2a (patch) | |
tree | afaddd5cd1679a6122659a741de7f8b217785e74 /ld | |
parent | 974eac9d7694ca14dcdf6d1a74777a265fffdb95 (diff) | |
download | gdb-ead3d5427a2df5e33316d4ad045510c1d2078c2a.zip gdb-ead3d5427a2df5e33316d4ad045510c1d2078c2a.tar.gz gdb-ead3d5427a2df5e33316d4ad045510c1d2078c2a.tar.bz2 |
Properly check symbol defined by assignment in linker script
Symbol defined by a linker assignment may have type bfd_link_hash_new
or bfd_link_hash_undefined. And h->def_regular is always set.
elf_i386_convert_load and elf_x86_64_convert_load should check
h->def_regular as well as bfd_link_hash_undefined and bfd_link_hash_new
to see if a symbol is defined by a linker script.
bfd/
PR ld/19319
* elf32-i386.c (elf_i386_convert_load): Check h->def_regular
instead of bfd_link_hash_new.
* elf64-x86-64.c (elf_x86_64_convert_load): Likewise. Skip
relocation overflow for bfd_link_hash_undefined and
bfd_link_hash_new if h->def_regular is set.
ld/testsuite/
PR ld/19319
* ld-i386/i386.exp: Run pr19319 test.
* ld-x86-64/x86-64.exp: Likewise.
* ld-i386/pr19319.dd: New file.
* ld-i386/pr19319a.S: Likewise.
* ld-i386/pr19319b.S: Likewise.
* ld-x86-64/pr19319.dd: Likewise.
* ld-x86-64/pr19319a.S: Likewise.
* ld-x86-64/pr19319b.S: Likewise.
Diffstat (limited to 'ld')
-rw-r--r-- | ld/testsuite/ChangeLog | 12 | ||||
-rw-r--r-- | ld/testsuite/ld-i386/i386.exp | 16 | ||||
-rw-r--r-- | ld/testsuite/ld-i386/pr19319.dd | 4 | ||||
-rw-r--r-- | ld/testsuite/ld-i386/pr19319a.S | 11 | ||||
-rw-r--r-- | ld/testsuite/ld-i386/pr19319b.S | 11 | ||||
-rw-r--r-- | ld/testsuite/ld-x86-64/pr19319.dd | 4 | ||||
-rw-r--r-- | ld/testsuite/ld-x86-64/pr19319a.S | 11 | ||||
-rw-r--r-- | ld/testsuite/ld-x86-64/pr19319b.S | 11 | ||||
-rw-r--r-- | ld/testsuite/ld-x86-64/x86-64.exp | 16 |
9 files changed, 96 insertions, 0 deletions
diff --git a/ld/testsuite/ChangeLog b/ld/testsuite/ChangeLog index 739f09f..d7ef3c4 100644 --- a/ld/testsuite/ChangeLog +++ b/ld/testsuite/ChangeLog @@ -1,3 +1,15 @@ +2015-12-01 H.J. Lu <hongjiu.lu@intel.com> + + PR ld/19319 + * ld-i386/i386.exp: Run pr19319 test. + * ld-x86-64/x86-64.exp: Likewise. + * ld-i386/pr19319.dd: New file. + * ld-i386/pr19319a.S: Likewise. + * ld-i386/pr19319b.S: Likewise. + * ld-x86-64/pr19319.dd: Likewise. + * ld-x86-64/pr19319a.S: Likewise. + * ld-x86-64/pr19319b.S: Likewise. + 2015-11-11 Alan Modra <amodra@gmail.com> Peter Bergner <bergner@vnet.ibm.com> diff --git a/ld/testsuite/ld-i386/i386.exp b/ld/testsuite/ld-i386/i386.exp index 84a63e8..a6601ed 100644 --- a/ld/testsuite/ld-i386/i386.exp +++ b/ld/testsuite/ld-i386/i386.exp @@ -537,6 +537,22 @@ if { [isnative] {{objdump {-dw} got1.dd}} \ "got1" \ ] \ + [list \ + "Build pr19319.so" \ + "-shared" \ + "" \ + { pr19319a.S } \ + "" \ + "pr19319.so" \ + ] \ + [list \ + "Build pr19319" \ + "-pie -nostdlib -nostartfiles tmpdir/pr19319.so" \ + "" \ + { pr19319b.S } \ + {{objdump {-dw} pr19319.dd}} \ + "pr19319" \ + ] \ ] run_ld_link_exec_tests [] [list \ diff --git a/ld/testsuite/ld-i386/pr19319.dd b/ld/testsuite/ld-i386/pr19319.dd new file mode 100644 index 0000000..f2593da --- /dev/null +++ b/ld/testsuite/ld-i386/pr19319.dd @@ -0,0 +1,4 @@ +#... +[a-f0-9]+ <_start>: +[ ]*[a-f0-9]+: 8d 81 ([0-9a-f]{2} ){4} * lea 0x[a-f0-9]+\(%ecx\),%eax +#pass diff --git a/ld/testsuite/ld-i386/pr19319a.S b/ld/testsuite/ld-i386/pr19319a.S new file mode 100644 index 0000000..04082be --- /dev/null +++ b/ld/testsuite/ld-i386/pr19319a.S @@ -0,0 +1,11 @@ + .text + .globl foo + .type foo, @function +foo: + movl __start_my_section@GOT(%ecx), %eax + ret + .size foo, .-foo + .section my_section,"aw",@progbits + .align 8 + .long 4660 + .long 22136 diff --git a/ld/testsuite/ld-i386/pr19319b.S b/ld/testsuite/ld-i386/pr19319b.S new file mode 100644 index 0000000..98294f6 --- /dev/null +++ b/ld/testsuite/ld-i386/pr19319b.S @@ -0,0 +1,11 @@ + .text + .globl _start + .type _start, @function +_start: + movl __start_my_section@GOT(%ecx), %eax + ret + .size _start, .-_start + .section my_section,"aw",@progbits + .align 8 + .long 4660 + .long 22136 diff --git a/ld/testsuite/ld-x86-64/pr19319.dd b/ld/testsuite/ld-x86-64/pr19319.dd new file mode 100644 index 0000000..9d6ffd9 --- /dev/null +++ b/ld/testsuite/ld-x86-64/pr19319.dd @@ -0,0 +1,4 @@ +#... +[a-f0-9]+ <_start>: +[ ]*[a-f0-9]+: 48 8d 05 ([0-9a-f]{2} ){4} * lea 0x[a-f0-9]+\(%rip\),%rax # [a-f0-9]+ <__start_my_section> +#pass diff --git a/ld/testsuite/ld-x86-64/pr19319a.S b/ld/testsuite/ld-x86-64/pr19319a.S new file mode 100644 index 0000000..438d7a5 --- /dev/null +++ b/ld/testsuite/ld-x86-64/pr19319a.S @@ -0,0 +1,11 @@ + .text + .globl foo + .type foo, @function +foo: + movq __start_my_section@GOTPCREL(%rip), %rax + ret + .size foo, .-foo + .section my_section,"aw",@progbits + .align 8 + .long 4660 + .long 22136 diff --git a/ld/testsuite/ld-x86-64/pr19319b.S b/ld/testsuite/ld-x86-64/pr19319b.S new file mode 100644 index 0000000..c9285c0 --- /dev/null +++ b/ld/testsuite/ld-x86-64/pr19319b.S @@ -0,0 +1,11 @@ + .text + .globl _start + .type _start, @function +_start: + movq __start_my_section@GOTPCREL(%rip), %rax + ret + .size _start, .-_start + .section my_section,"aw",@progbits + .align 8 + .long 4660 + .long 22136 diff --git a/ld/testsuite/ld-x86-64/x86-64.exp b/ld/testsuite/ld-x86-64/x86-64.exp index 63d7211..f0f47eb 100644 --- a/ld/testsuite/ld-x86-64/x86-64.exp +++ b/ld/testsuite/ld-x86-64/x86-64.exp @@ -546,6 +546,22 @@ if { [isnative] && [which $CC] != 0 } { {{objdump {-dw} gotpcrel1.dd}} \ "gotpcrel1" \ ] \ + [list \ + "Build pr19319.so" \ + "-shared" \ + "" \ + { pr19319a.S } \ + "" \ + "pr19319.so" \ + ] \ + [list \ + "Build pr19319" \ + "-pie -nostdlib -nostartfiles tmpdir/pr19319.so" \ + "" \ + { pr19319b.S } \ + {{objdump {-dw} pr19319.dd}} \ + "pr19319" \ + ] \ ] run_ld_link_exec_tests [] [list \ |