From d0240f628cb407bab0bd45aba7a8423f9be96fa6 Mon Sep 17 00:00:00 2001 From: DJ Delorie Date: Wed, 31 Aug 2005 23:45:46 +0000 Subject: [PATCH] * elf32-i386.c (elf_i386_check_relocs): Don't cast a unary & address operator, as that breaks GCC's strict aliasing rules. (elf_i386_size_dynamic_sections): Avoid the need for type punning. * elf64-x86-64.c (elf_x86_64_check_relocs): Don't cast a unary & address operator, as that breaks GCC's strict aliasing rules. (elf_x86_64_size_dynamic_sections): Avoid the need for type punning. --- bfd/ChangeLog | 12 ++++++++++++ bfd/elf32-i386.c | 9 +++++---- bfd/elf64-x86-64.c | 11 +++++++---- 3 files changed, 24 insertions(+), 8 deletions(-) diff --git a/bfd/ChangeLog b/bfd/ChangeLog index 6746e0786..33f5f5589 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,15 @@ +2005-08-31 DJ Delorie + + * elf32-i386.c (elf_i386_check_relocs): Don't cast a unary & + address operator, as that breaks GCC's strict aliasing rules. + (elf_i386_size_dynamic_sections): Avoid the need for type + punning. + * elf64-x86-64.c (elf_x86_64_check_relocs): Don't cast a unary + & address operator, as that breaks GCC's strict aliasing + rules. + (elf_x86_64_size_dynamic_sections): Avoid the need for type + punning. + 2005-08-30 Phil Edwards * config.bfd (i[3-7]86-*-vxworks): Match vxworks* instead. diff --git a/bfd/elf32-i386.c b/bfd/elf32-i386.c index 78c4fa346..acb9f1936 100644 --- a/bfd/elf32-i386.c +++ b/bfd/elf32-i386.c @@ -1165,6 +1165,7 @@ elf_i386_check_relocs (bfd *abfd, } else { + void **vpp; /* Track dynamic relocs needed for local syms too. We really need local syms available to do this easily. Oh well. */ @@ -1175,8 +1176,8 @@ elf_i386_check_relocs (bfd *abfd, if (s == NULL) return FALSE; - head = ((struct elf_i386_dyn_relocs **) - &elf_section_data (s)->local_dynrel); + vpp = &elf_section_data (s)->local_dynrel; + head = (struct elf_i386_dyn_relocs **)vpp; } p = *head; @@ -1823,8 +1824,8 @@ elf_i386_size_dynamic_sections (bfd *output_bfd ATTRIBUTE_UNUSED, { struct elf_i386_dyn_relocs *p; - for (p = *((struct elf_i386_dyn_relocs **) - &elf_section_data (s)->local_dynrel); + for (p = ((struct elf_i386_dyn_relocs *) + elf_section_data (s)->local_dynrel); p != NULL; p = p->next) { diff --git a/bfd/elf64-x86-64.c b/bfd/elf64-x86-64.c index 4c7a49e3a..0520906b2 100644 --- a/bfd/elf64-x86-64.c +++ b/bfd/elf64-x86-64.c @@ -946,6 +946,7 @@ elf64_x86_64_check_relocs (bfd *abfd, struct bfd_link_info *info, asection *sec, } else { + void **vpp; /* Track dynamic relocs needed for local syms too. We really need local syms available to do this easily. Oh well. */ @@ -956,8 +957,10 @@ elf64_x86_64_check_relocs (bfd *abfd, struct bfd_link_info *info, asection *sec, if (s == NULL) return FALSE; - head = ((struct elf64_x86_64_dyn_relocs **) - &elf_section_data (s)->local_dynrel); + /* Beware of type punned pointers vs strict aliasing + rules. */ + vpp = &(elf_section_data (s)->local_dynrel); + head = (struct elf64_x86_64_dyn_relocs **)vpp; } p = *head; @@ -1586,8 +1589,8 @@ elf64_x86_64_size_dynamic_sections (bfd *output_bfd ATTRIBUTE_UNUSED, { struct elf64_x86_64_dyn_relocs *p; - for (p = *((struct elf64_x86_64_dyn_relocs **) - &elf_section_data (s)->local_dynrel); + for (p = (struct elf64_x86_64_dyn_relocs *) + (elf_section_data (s)->local_dynrel); p != NULL; p = p->next) { -- 2.11.4.GIT