diff options
Diffstat (limited to 'lld/ELF/InputFiles.cpp')
| -rw-r--r-- | lld/ELF/InputFiles.cpp | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/lld/ELF/InputFiles.cpp b/lld/ELF/InputFiles.cpp index 35626092349..e8f2beed5d0 100644 --- a/lld/ELF/InputFiles.cpp +++ b/lld/ELF/InputFiles.cpp @@ -377,17 +377,22 @@ template <class ELFT> InputSectionBase<ELFT> * elf::ObjectFile<ELFT>::getSection(const Elf_Sym &Sym) const { uint32_t Index = this->getSectionIndex(Sym); - if (Index == 0) - return nullptr; if (Index >= Sections.size()) fatal(getFilename(this) + ": invalid section index: " + Twine(Index)); InputSectionBase<ELFT> *S = Sections[Index]; + // We found that GNU assembler 2.17.50 [FreeBSD] 2007-07-03 // could generate broken objects. STT_SECTION symbols can be // associated with SHT_REL[A]/SHT_SYMTAB/SHT_STRTAB sections. // In this case it is fine for section to be null here as we // do not allocate sections of these types. - if (!S || S == &InputSectionBase<ELFT>::Discarded) + if (!S) { + if (Index == 0 || Sym.getType() == STT_SECTION) + return nullptr; + fatal(getFilename(this) + ": invalid section index: " + Twine(Index)); + } + + if (S == &InputSectionBase<ELFT>::Discarded) return S; return S->Repl; } |

