aboutsummaryrefslogtreecommitdiff
path: root/bfd
diff options
context:
space:
mode:
authorH.J. Lu <hjl.tools@gmail.com>2016-06-15 18:42:56 -0700
committerH.J. Lu <hjl.tools@gmail.com>2016-06-15 18:42:56 -0700
commit4c23f2ffa8fbd467bca51956130a1f30cfe34371 (patch)
tree1eaaf047843ab31401b50124d66d4c24f6583e60 /bfd
parent77b1142d4e709cb12a460bdfad6f109d67260457 (diff)
downloadgdb-4c23f2ffa8fbd467bca51956130a1f30cfe34371.zip
gdb-4c23f2ffa8fbd467bca51956130a1f30cfe34371.tar.gz
gdb-4c23f2ffa8fbd467bca51956130a1f30cfe34371.tar.bz2
Check SEC_ALLOC before allocating dynamic relocation
* elf32-i386.c (elf_i386_check_relocs): Check SEC_ALLOC before allocating dynamic relocation. * elf64-x86-64.c (elf_x86_64_check_relocs): Likewise.
Diffstat (limited to 'bfd')
-rw-r--r--bfd/ChangeLog6
-rw-r--r--bfd/elf32-i386.c27
-rw-r--r--bfd/elf64-x86-64.c27
3 files changed, 32 insertions, 28 deletions
diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index 38d28f3..344e08b 100644
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,9 @@
+2016-06-15 H.J. Lu <hongjiu.lu@intel.com>
+
+ * elf32-i386.c (elf_i386_check_relocs): Check SEC_ALLOC before
+ allocating dynamic relocation.
+ * elf64-x86-64.c (elf_x86_64_check_relocs): Likewise.
+
2016-06-14 Senthil Kumar Selvaraj <senthil_kumar.selvaraj@atmel.com>
PR ld/20254
diff --git a/bfd/elf32-i386.c b/bfd/elf32-i386.c
index 686c068..793fb39 100644
--- a/bfd/elf32-i386.c
+++ b/bfd/elf32-i386.c
@@ -2202,20 +2202,19 @@ do_size:
may need to keep relocations for symbols satisfied by a
dynamic library if we manage to avoid copy relocs for the
symbol. */
- if ((bfd_link_pic (info)
- && (sec->flags & SEC_ALLOC) != 0
- && (r_type != R_386_PC32
- || (h != NULL
- && (! (bfd_link_pie (info)
- || SYMBOLIC_BIND (info, h))
- || h->root.type == bfd_link_hash_defweak
- || !h->def_regular))))
- || (ELIMINATE_COPY_RELOCS
- && !bfd_link_pic (info)
- && (sec->flags & SEC_ALLOC) != 0
- && h != NULL
- && (h->root.type == bfd_link_hash_defweak
- || !h->def_regular)))
+ if ((sec->flags & SEC_ALLOC) != 0
+ && ((bfd_link_pic (info)
+ && (r_type != R_386_PC32
+ || (h != NULL
+ && (! (bfd_link_pie (info)
+ || SYMBOLIC_BIND (info, h))
+ || h->root.type == bfd_link_hash_defweak
+ || !h->def_regular))))
+ || (ELIMINATE_COPY_RELOCS
+ && !bfd_link_pic (info)
+ && h != NULL
+ && (h->root.type == bfd_link_hash_defweak
+ || !h->def_regular))))
{
struct elf_dyn_relocs *p;
struct elf_dyn_relocs **head;
diff --git a/bfd/elf64-x86-64.c b/bfd/elf64-x86-64.c
index b209604..8c8bcce 100644
--- a/bfd/elf64-x86-64.c
+++ b/bfd/elf64-x86-64.c
@@ -2601,20 +2601,19 @@ do_size:
may need to keep relocations for symbols satisfied by a
dynamic library if we manage to avoid copy relocs for the
symbol. */
- if ((bfd_link_pic (info)
- && (sec->flags & SEC_ALLOC) != 0
- && (! IS_X86_64_PCREL_TYPE (r_type)
- || (h != NULL
- && (! (bfd_link_pie (info)
- || SYMBOLIC_BIND (info, h))
- || h->root.type == bfd_link_hash_defweak
- || !h->def_regular))))
- || (ELIMINATE_COPY_RELOCS
- && !bfd_link_pic (info)
- && (sec->flags & SEC_ALLOC) != 0
- && h != NULL
- && (h->root.type == bfd_link_hash_defweak
- || !h->def_regular)))
+ if ((sec->flags & SEC_ALLOC) != 0
+ && ((bfd_link_pic (info)
+ && (! IS_X86_64_PCREL_TYPE (r_type)
+ || (h != NULL
+ && (! (bfd_link_pie (info)
+ || SYMBOLIC_BIND (info, h))
+ || h->root.type == bfd_link_hash_defweak
+ || !h->def_regular))))
+ || (ELIMINATE_COPY_RELOCS
+ && !bfd_link_pic (info)
+ && h != NULL
+ && (h->root.type == bfd_link_hash_defweak
+ || !h->def_regular))))
{
struct elf_dyn_relocs *p;
struct elf_dyn_relocs **head;