espressif/binutils-gdb
1
0
Fork 0
mirror of https://github.com/espressif/binutils-gdb.git synced 2025-08-01 08:54:44 +08:00
Code Issues Packages Projects Releases Wiki Activity

PR22127, as segfaults assembling invalid .reloc

Browse Source 
"sec" gets set to NULL on errors in the offset expression.  This patch
disables part of the reloc expression processing that needs "sec"
valid.  I didn't disable the entire reloc expression handling so that
errors in the reloc expression are reported even when the offset
expression has an error.

	PR 22127
	* write.c (resolve_reloc_expr_symbols): Don't segfault when
	sec has been set to NULL.
This commit is contained in:
Alan Modra
2017-09-14 10:19:31 +09:30
parent 0952b7101e
commit 2578f2f307
2 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
gas/ChangeLog
View File

@ -1,3 +1,9 @@
2017-09-14 Alan Modra <amodra@gmail.com>
PR 22127
* write.c (resolve_reloc_expr_symbols): Don't segfault when
sec has been set to NULL.
2017-09-09 H.J. Lu <hongjiu.lu@intel.com> 2017-09-09 H.J. Lu <hongjiu.lu@intel.com>
* config/tc-i386.c (NOTRACK_PREFIX): Removed. * config/tc-i386.c (NOTRACK_PREFIX): Removed.

2
gas/write.c
View File

@ -724,7 +724,7 @@ resolve_reloc_expr_symbols (void)
as_bad_where (r->file, r->line, _("invalid reloc expression")); as_bad_where (r->file, r->line, _("invalid reloc expression"));
sec = NULL; sec = NULL;
} }
else if (sym != NULL) else if (sym != NULL && sec != NULL)
{ {
/* Convert relocs against local symbols to refer to the /* Convert relocs against local symbols to refer to the
corresponding section symbol plus offset instead. Keep corresponding section symbol plus offset instead. Keep