From 91f781ffbbd5b4eab6aa609a027a5b6878cf701e Mon Sep 17 00:00:00 2001 From: Fred Fish Date: Wed, 17 Jun 1992 20:58:46 +0000 Subject: [PATCH] * elf.c (bfd_section_from_shdr, elf_slurp_symbol_table): Correct misconception that there can be only one symbol table. Only call elf_slurp_symbol_table on the full symbol table, not the dynamic one which is only a subset of the full one. (The bfd library was slurping in the first ELF symbol table it found and then ignoring any others. For dynamically linked executables, this happened to be the dynamic symbol table, which is a subset of the full symbol table and only contains the symbols that need to be used in dynamic linking.) --- bfd/ChangeLog | 7 +++++++ bfd/elf.c | 19 ++++++++++++++++--- 2 files changed, 23 insertions(+), 3 deletions(-) diff --git a/bfd/ChangeLog b/bfd/ChangeLog index 78d0c98..7cbfe73 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,10 @@ +Wed Jun 17 13:55:31 1992 Fred Fish (fnf@cygnus.com) + + * elf.c (bfd_section_from_shdr, elf_slurp_symbol_table): + Correct misconception that there can be only one symbol table. + Only call elf_slurp_symbol_table on the full symbol table, not + the dynamic one which is only a subset of the full one. + Wed Jun 17 15:54:57 1992 Ken Raeburn (Raeburn@Cygnus.COM) * coffcode.h (coff_compute_section_file_positions): For I960, diff --git a/bfd/elf.c b/bfd/elf.c index fe4d37c..c41ba0a 100644 --- a/bfd/elf.c +++ b/bfd/elf.c @@ -466,6 +466,11 @@ DEFUN(bfd_section_from_shdr, (abfd, shindex), /* *these* do a lot of work -- but build no sections! */ /* the spec says there can be multiple strtabs, but only one symtab */ /* but there can be lots of REL* sections. */ + /* FIXME: The above statement is wrong! There are typically at least + two symbol tables in a dynamically linked executable, ".dynsym" + which is the dynamic linkage symbol table and ".symtab", which is + the "traditional" symbol table. -fnf */ + { asection *target_sect; unsigned int idx; @@ -474,8 +479,13 @@ DEFUN(bfd_section_from_shdr, (abfd, shindex), bfd_section_from_shdr (abfd, hdr->sh_info); /* target */ target_sect = section_from_elf_index (abfd, hdr->sh_info); +#if 0 + /* FIXME: We are only prepared to read one symbol table, so + do NOT read the dynamic symbol table since it is only a + subset of the full symbol table. Also see comment above. -fnf */ if (!elf_slurp_symbol_table(abfd, i_shdrp + hdr->sh_link)) return false; +#endif target_sect->reloc_count = hdr->sh_size / hdr->sh_entsize; target_sect->flags |= SEC_RELOC; @@ -1613,8 +1623,6 @@ DEFUN (elf_compute_section_file_positions, (abfd), bfd *abfd) i_ehdrp->e_entry = bfd_get_start_address (abfd); i_ehdrp->e_shentsize = sizeof (Elf_External_Shdr); - /* can't do this: we'll need many more... */ - /* i_ehdr.e_shnum = bfd_count_sections(abfd)+1; /* include 0th, shstrtab */ /* figure at most each section can have a rel, strtab, symtab */ maxsections = 4*bfd_count_sections(abfd)+2; @@ -1908,6 +1916,11 @@ DEFUN (elf_slurp_symbol_table, (abfd, hdr), unsigned int *table_ptr; /* bfd symbol translation table */ /* this is only valid because there is only one symtab... */ + /* FIXME: This is incorrect, there may also be a dynamic symbol + table which is a subset of the full symbol table. We either need + to be prepared to read both (and merge them) or ensure that we + only read the full symbol table. Currently we only get called to + read the full symbol table. -fnf */ if (bfd_get_outsymbols (abfd) != NULL) { return (true); @@ -2051,7 +2064,7 @@ DEFUN (elf_get_symtab_upper_bound, (abfd), bfd *abfd) { unsigned int symtab_size = 0; - /* if (elf_slurp_symbol_table (abfd, ???)) */ + /* if (elf_slurp_symbol_table (abfd, FIXME...)) */ { symtab_size = (bfd_get_symcount (abfd) + 1) * (sizeof (asymbol)); } -- 2.7.4