From: Nikita Ermakov <coffe92@gmail.com>
Date: Thu, 14 May 2020 22:57:57 +0000 (+0930)
Subject: gold: powerpc: Test whether sym is not a plugin in do_gc_mark_symbol
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=6a31512fd4fdd38ee2b1ea78543d352f961ca172;p=binutils-gdb.git

gold: powerpc: Test whether sym is not a plugin in do_gc_mark_symbol

sym->object() could be either a Plugin or Powerpc_relobj. There could
be a situation when Pluginobj would be proccessed in
ppc_object->get_opd_ent(dst_off) as Powerpc_relobj and it leads to the
segmentation fault.

	* powerpc.cc (do_gc_mark_symbol): Don't segfault on plugin symbols.
---

diff --git a/gold/ChangeLog b/gold/ChangeLog
index 212c9fac9da..e85669a9f4b 100644
--- a/gold/ChangeLog
+++ b/gold/ChangeLog
@@ -1,3 +1,7 @@
+2020-05-15  Nikita Ermakov  <coffe92@gmail.com>
+
+	* powerpc.cc (do_gc_mark_symbol): Don't segfault on plugin symbols.
+
 2020-05-14  Nick Clifton  <nickc@redhat.com>
 
 	* po/sr.po: New Serbian translation.
diff --git a/gold/powerpc.cc b/gold/powerpc.cc
index eae483212b8..2010c1e3d2b 100644
--- a/gold/powerpc.cc
+++ b/gold/powerpc.cc
@@ -9056,7 +9056,7 @@ Target_powerpc<size, big_endian>::do_gc_mark_symbol(
     Symbol_table* symtab,
     Symbol* sym) const
 {
-  if (size == 64)
+  if (size == 64 && sym->object()->pluginobj() == NULL)
     {
       Powerpc_relobj<size, big_endian>* ppc_object
 	= static_cast<Powerpc_relobj<size, big_endian>*>(sym->object());