[PATCH] Fix Alpha FAIL: bootstrap with --relax
Jakub Jelinek
jakub@redhat.com
Thu Feb 6 16:10:00 GMT 2003
Hi!
During bootstrap with relax, archive.o contains a bunch of R_ALPHA_LIT*
relocs but only against global syms and then some R_ALPHA_GPREL*
against local syms.
This means that local_got_entries is not allocated (all NEED_GOT_ENTRY
end up in some h->got_entries slots). When elf64_alpha_relax_sections
looks at one of the R_ALPHA_GPREL* relocs, r_symndx is local,
info.first_gotent gets set to
&((struct alpha_elf_got_entry **)NULL)[r_symndx]
and segfaults a few lines later.
Dunno if it is better to do it as below or instead set first_gotent
to NULL and handle first_gotent == NULL specially where it is used.
2003-02-06 Jakub Jelinek <jakub@redhat.com>
* elf64-alpha.c (elf64_alpha_relax_section): Don't crash if
local_got_entries is NULL.
--- bfd/elf64-alpha.c.jj 2003-02-06 10:53:56.000000000 -0500
+++ bfd/elf64-alpha.c 2003-02-06 10:53:56.000000000 -0500
@@ -2184,7 +2184,13 @@ elf64_alpha_relax_section (abfd, sec, li
info.h = NULL;
info.other = isym->st_other;
- info.first_gotent = &local_got_entries[r_symndx];
+ if (local_got_entries)
+ info.first_gotent = &local_got_entries[r_symndx];
+ else
+ {
+ info.first_gotent = &info.gotent;
+ info.gotent = NULL;
+ }
}
else
{
Jakub
More information about the Binutils
mailing list