OpenE2K
/
binutils-gdb
12
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

PR24337, segfault in _bfd_elf_rela_local_sym, again 

Reverts commit 1ff31e135f, fixing the problem more generally.  There
are likely other place that will segfault on a NULL section.

	PR 24337
	* elf.c (_bfd_elf_rela_local_sym): Revert last change.
	(_bfd_elf_rel_local_sym): Likewise.
	* elflink.c (elf_link_input_bfd): Use bfd_und_section for
	section of symbols with unrecognized shndx.
This commit is contained in:
Alan Modra 2019-03-16 09:34:47 +10:30
parent fdc01131eb
commit 6835821be8
3 changed files with 14 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
bfd/ChangeLog
View File

@ -1,3 +1,11 @@
2019-03-16 Alan Modra <amodra@gmail.com>
PR 24337
* elf.c (_bfd_elf_rela_local_sym): Revert last change.
(_bfd_elf_rel_local_sym): Likewise.
* elflink.c (elf_link_input_bfd): Use bfd_und_section for
section of symbols with unrecognized shndx.
2019-03-15 H.J. Lu <hongjiu.lu@intel.com>
PR ld/24267

9
bfd/elf.c
View File

@ -11822,10 +11822,9 @@ _bfd_elf_rela_local_sym (bfd *abfd,
asection *sec = *psec;
bfd_vma relocation;
relocation = sym->st_value;
if (sec == NULL)
return relocation;
relocation += sec->output_section->vma + sec->output_offset;
relocation = (sec->output_section->vma
+ sec->output_offset
+ sym->st_value);
if ((sec->flags & SEC_MERGE)
&& ELF_ST_TYPE (sym->st_info) == STT_SECTION
&& sec->sec_info_type == SEC_INFO_TYPE_MERGE)
@ -11859,7 +11858,7 @@ _bfd_elf_rel_local_sym (bfd *abfd,
{
asection *sec = *psec;
if (sec == NULL || sec->sec_info_type != SEC_INFO_TYPE_MERGE)
if (sec->sec_info_type != SEC_INFO_TYPE_MERGE)
return sym->st_value + addend;
return _bfd_merged_section_offset (abfd, psec,

5
bfd/elflink.c
View File

@ -4446,7 +4446,7 @@ error_free_dyn:
continue;
/* If we aren't prepared to handle locals within the globals
then we'll likely segfault on a NULL section. */
then we'll likely segfault on a NULL section. */
bfd_set_error (bfd_error_bad_value);
goto error_free_vers;
@ -10421,8 +10421,7 @@ elf_link_input_bfd (struct elf_final_link_info *flinfo, bfd *input_bfd)
{
/* Don't attempt to output symbols with st_shnx in the
reserved range other than SHN_ABS and SHN_COMMON. */
*ppsection = NULL;
continue;
isec = bfd_und_section_ptr;
}
else if (isec->sec_info_type == SEC_INFO_TYPE_MERGE
&& ELF_ST_TYPE (isym->st_info) != STT_SECTION)