PR29355, ld segfaults with -r/-q and custom-named section .rela*
authorAlan Modra <amodra@gmail.com>
Tue, 12 Jul 2022 01:21:52 +0000 (10:51 +0930)
committerAlan Modra <amodra@gmail.com>
Tue, 12 Jul 2022 02:35:28 +0000 (12:05 +0930)
commit35c5dcc666082f0edf6dd559e46b98d0e7986a2e
tree67114ad75828d4199c8115eb9fa4b86edfa8a9e9
parentd5ffc11fcbd43cb72e3ba55a52bb8df6374a8745
PR29355, ld segfaults with -r/-q and custom-named section .rela*

The bug testcase uses an output section named .rel or .rela which has
input .data sections mapped to it.  The input .data section has
relocations.  When counting output relocations SHT_REL and SHT_RELA
section reloc_count is ignored, with the justification that reloc
sections themselves can't have relocations and some backends use
reloc_count in reloc sections.  However, the test wrongly used the
output section type (which normally would match input section type).
Fix that.  Note that it is arguably wrong for ld to leave the output
.rel/.rela section type as SHT_REL/SHT_RELA when non-empty non-reloc
sections are written to it, but I'm not going to change that since it
might be useful to hand-craft relocs in a data section that is then
written to a SHT_REL/SHT_RELA output section.

PR 29355
* elflink.c (bfd_elf_final_link): Use input section type rather
than output section type to determine whether to exclude using
reloc_count from that section.
bfd/elflink.c