diff options
author | Alan Modra <amodra@bigpond.net.au> | 2002-03-13 07:26:24 +0000 |
---|---|---|
committer | Alan Modra <amodra@bigpond.net.au> | 2002-03-13 07:26:24 +0000 |
commit | 886e40b5a39601bcd72e2e2dbe20ac11a2186bba (patch) | |
tree | 39678b846b38f6f641d3d4797ee6ed4b5e5688fc /bfd | |
parent | d22993a59eb86df02f312948ab99b92c6deeacb4 (diff) | |
download | gdb-886e40b5a39601bcd72e2e2dbe20ac11a2186bba.tar.gz |
Revert last change.
Diffstat (limited to 'bfd')
-rw-r--r-- | bfd/ChangeLog | 4 | ||||
-rw-r--r-- | bfd/elflink.h | 96 |
2 files changed, 43 insertions, 57 deletions
diff --git a/bfd/ChangeLog b/bfd/ChangeLog index 3416865fd69..463adc01a80 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,9 +1,5 @@ 2002-03-13 Alan Modra <amodra@bigpond.net.au> - * elflink.h (NAME(bfd_elf,size_dynamic_sections)): Use ELF section - types instead of section names to determine whether init/fini array - dynamic tags are needed. - * elflink.h: Formatting fixes. (elf_link_output_extsym): Merge undefined and undef weak cases. diff --git a/bfd/elflink.h b/bfd/elflink.h index 438cb4da30d..3e8551f9a38 100644 --- a/bfd/elflink.h +++ b/bfd/elflink.h @@ -3069,9 +3069,6 @@ NAME(bfd_elf,size_dynamic_sections) (output_bfd, soname, rpath, struct elf_info_failed eif; struct elf_link_hash_entry *h; asection *dynstr; - bfd *sub; - asection *o; - int dt_done; *sinterpptr = bfd_get_section_by_name (dynobj, ".interp"); BFD_ASSERT (*sinterpptr != NULL || info->shared); @@ -3203,58 +3200,51 @@ NAME(bfd_elf,size_dynamic_sections) (output_bfd, soname, rpath, return false; } - dt_done = 0; - for (sub = info->input_bfds; sub != NULL; sub = sub->link_next) - for (o = sub->sections; o != NULL; o = o->next) - { - int tag1, tag2; - - if (o->output_section == bfd_abs_section_ptr) - continue; - - switch (elf_section_data (o)->this_hdr.sh_type) - { - default: - continue; - - case SHT_INIT_ARRAY: - if (dt_done & 1) - continue; - dt_done |= 1; - tag1 = DT_INIT_ARRAY; - tag2 = DT_INIT_ARRAYSZ; - break; - - case SHT_FINI_ARRAY: - if (dt_done & 2) - continue; - dt_done |= 2; - tag1 = DT_FINI_ARRAY; - tag2 = DT_FINI_ARRAYSZ; - break; - - case SHT_PREINIT_ARRAY: - /* DT_PREINIT_ARRAY is not allowed in a shared library. */ - if (info->shared) - { - (*_bfd_error_handler) - (_("%s: .preinit_array section is not allowed in DSO"), - bfd_archive_filename (sub)); - bfd_set_error (bfd_error_bad_value); - return false; - } - if (dt_done & 4) - continue; - dt_done |= 4; - tag1 = DT_PREINIT_ARRAY; - tag2 = DT_PREINIT_ARRAYSZ; - break; - } + if (bfd_get_section_by_name (output_bfd, ".preinit_array") != NULL) + { + /* DT_PREINIT_ARRAY is not allowed in shared library. */ + if (info->shared) + { + bfd *sub; + asection *o; + + for (sub = info->input_bfds; sub != NULL; + sub = sub->link_next) + for (o = sub->sections; o != NULL; o = o->next) + if (elf_section_data (o)->this_hdr.sh_type + == SHT_PREINIT_ARRAY) + { + (*_bfd_error_handler) + (_("%s: .preinit_array section is not allowed in DSO"), + bfd_archive_filename (sub)); + break; + } - if (!elf_add_dynamic_entry (info, (bfd_vma) tag1, (bfd_vma) 0) - || !elf_add_dynamic_entry (info, (bfd_vma) tag2, (bfd_vma) 0)) return false; - } + } + + if (!elf_add_dynamic_entry (info, (bfd_vma) DT_PREINIT_ARRAY, + (bfd_vma) 0) + || !elf_add_dynamic_entry (info, (bfd_vma) DT_PREINIT_ARRAYSZ, + (bfd_vma) 0)) + return false; + } + if (bfd_get_section_by_name (output_bfd, ".init_array") != NULL) + { + if (!elf_add_dynamic_entry (info, (bfd_vma) DT_INIT_ARRAY, + (bfd_vma) 0) + || !elf_add_dynamic_entry (info, (bfd_vma) DT_INIT_ARRAYSZ, + (bfd_vma) 0)) + return false; + } + if (bfd_get_section_by_name (output_bfd, ".fini_array") != NULL) + { + if (!elf_add_dynamic_entry (info, (bfd_vma) DT_FINI_ARRAY, + (bfd_vma) 0) + || !elf_add_dynamic_entry (info, (bfd_vma) DT_FINI_ARRAYSZ, + (bfd_vma) 0)) + return false; + } dynstr = bfd_get_section_by_name (dynobj, ".dynstr"); /* If .dynstr is excluded from the link, we don't want any of |