[binutils-gdb] PR26475 UBSAN: elfxx-mips.c:12180 null pointer memset

Alan Modra amodra@sourceware.org
Wed Aug 26 13:55:01 GMT 2020


https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=55e61b8ad977cba772753c8dbba5c81f6b1fa54c

commit 55e61b8ad977cba772753c8dbba5c81f6b1fa54c
Author: Alan Modra <amodra@gmail.com>
Date:   Wed Aug 26 18:22:07 2020 +0930

    PR26475 UBSAN: elfxx-mips.c:12180 null pointer memset
    
    Another memset(0,0,0)
    
            PR 26475
            * elfxx-mips.c (_bfd_mips_elf_finish_dynamic_sections): Check
            sstubs->contents != NULL.

Diff:
---
 bfd/ChangeLog    | 6 ++++++
 bfd/elfxx-mips.c | 3 ++-
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index 20a02d9238f..3a85c3eb9a5 100644
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,9 @@
+2020-08-26  Alan Modra  <amodra@gmail.com>
+
+	PR 26475
+	* elfxx-mips.c (_bfd_mips_elf_finish_dynamic_sections): Check
+	sstubs->contents != NULL.
+
 2020-08-26  Alan Modra  <amodra@gmail.com>
 
 	PR 26453
diff --git a/bfd/elfxx-mips.c b/bfd/elfxx-mips.c
index fda653e1afc..bc0843508e0 100644
--- a/bfd/elfxx-mips.c
+++ b/bfd/elfxx-mips.c
@@ -12172,7 +12172,8 @@ _bfd_mips_elf_finish_dynamic_sections (bfd *output_bfd,
 					     s->contents));
 
 	    /* Clean up a dummy stub function entry in .text.  */
-	    if (htab->sstubs != NULL)
+	    if (htab->sstubs != NULL
+		&& htab->sstubs->contents != NULL)
 	      {
 		file_ptr dummy_offset;


More information about the Binutils-cvs mailing list