Fix an illegal memory access when parsing a corrupt ELF file.

	PR 24333
	* elflink.c (_bfd_elf_add_default_symbol): Add a check for a NULL
	section owner pointer when adding the default symbol.

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index 447eb707c877399943c8913826c88979dbf614dc..7ed3929c0aedc8d1a85302f7a500a871e2741116 100644 (file)
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,9 @@
+2019-03-14  Nick Clifton  <nickc@redhat.com>
+
+       PR 24333
+       * elflink.c (_bfd_elf_add_default_symbol): Add a check for a NULL
+       section owner pointer when adding the default symbol.
+
 2019-03-14  Nick Clifton  <nickc@redhat.com>
 
        PR 24332

diff --git a/bfd/elflink.c b/bfd/elflink.c
index 3ac58daafda812d1db60999691e47217c2c15621..3413726b94785260ff9af1271ec8514c489aaf76 100644 (file)
--- a/bfd/elflink.c
+++ b/bfd/elflink.c
@@ -1941,6 +1941,7 @@ _bfd_elf_add_default_symbol (bfd *abfd,
        {
          bh = &hi->root;
          if (bh->type == bfd_link_hash_defined
+             && bh->u.def.section->owner != NULL
              && (bh->u.def.section->owner->flags & BFD_PLUGIN) != 0)
            {
              /* Mark the previous definition from IR object as