From: Nick Clifton Date: Tue, 6 Dec 2016 16:26:42 +0000 (+0000) Subject: Fix seg-fault running strip on a corrupt binary. X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=8b929e420a810afe23d5ba57a98109c74d1d5816;p=binutils-gdb.git Fix seg-fault running strip on a corrupt binary. PR binutils/20930 * objcopy.c (mark_symbols_used_in_relocations): Check for a null symbol pointer pointer before attempting to mark the symbol as kept. --- diff --git a/binutils/ChangeLog b/binutils/ChangeLog index fdbb0564c09..b3429f7989c 100644 --- a/binutils/ChangeLog +++ b/binutils/ChangeLog @@ -1,3 +1,10 @@ +2016-12-06 Nick Clifton + + PR binutils/20930 + * objcopy.c (mark_symbols_used_in_relocations): Check for a null + symbol pointer pointer before attempting to mark the symbol as + kept. + 2016-12-05 Jose E. Marchesi * MAINTAINERS: Add myself as maintainer for the SPARC targets. diff --git a/binutils/objcopy.c b/binutils/objcopy.c index 6a398ce4dca..c78001204da 100644 --- a/binutils/objcopy.c +++ b/binutils/objcopy.c @@ -3551,8 +3551,9 @@ mark_symbols_used_in_relocations (bfd *ibfd, sec_ptr isection, void *symbolsarg) special bfd section symbols, then mark it with BSF_KEEP. */ for (i = 0; i < relcount; i++) { - /* See PR 20923 for a reproducer for the NULL test. */ + /* See PRs 20923 and 20930 for reproducers for the NULL tests. */ if (relpp[i]->sym_ptr_ptr != NULL + && * relpp[i]->sym_ptr_ptr != NULL && *relpp[i]->sym_ptr_ptr != bfd_com_section_ptr->symbol && *relpp[i]->sym_ptr_ptr != bfd_abs_section_ptr->symbol && *relpp[i]->sym_ptr_ptr != bfd_und_section_ptr->symbol)