sh_addralign inconsistent with sh_addr

The ELF gABI says in part of sh_addralign:  "The value of sh_addr must
be congruent to 0, modulo the value of sh_addralign."

	* elf.c (elf_fake_sections): Ensure sh_addralign is such that
	sh_addr mod sh_addalign is zero.
This commit is contained in:
Alan Modra 2020-03-04 15:26:00 +10:30
parent 1039fd9ac2
commit 233bf4f847
2 changed files with 10 additions and 1 deletions

View File

@ -1,3 +1,8 @@
2020-03-04 Alan Modra <amodra@gmail.com>
* elf.c (elf_fake_sections): Ensure sh_addralign is such that
sh_addr mod sh_addalign is zero.
2020-03-04 Alan Modra <amodra@gmail.com>
* format.c (bfd_check_format_matches): Call cleanup on error exit.

View File

@ -3192,6 +3192,7 @@ elf_fake_sections (bfd *abfd, asection *asect, void *fsarg)
unsigned int sh_type;
const char *name = asect->name;
bfd_boolean delay_st_name_p = FALSE;
bfd_vma mask;
if (arg->failed)
{
@ -3291,7 +3292,10 @@ elf_fake_sections (bfd *abfd, asection *asect, void *fsarg)
arg->failed = TRUE;
return;
}
this_hdr->sh_addralign = (bfd_vma) 1 << asect->alignment_power;
/* Set sh_addralign to the highest power of two given by alignment
consistent with the section VMA. Linker scripts can force VMA. */
mask = ((bfd_vma) 1 << asect->alignment_power) | this_hdr->sh_addr;
this_hdr->sh_addralign = mask & -mask;
/* The sh_entsize and sh_info fields may have been set already by
copy_private_section_data. */