Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 7 Sep 2012 18:03:29 +0000 (UTC)
From:      Dimitry Andric <dim@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-9@freebsd.org
Subject:   svn commit: r240208 - in stable: 7/contrib/binutils/binutils 8/contrib/binutils/binutils 9/contrib/binutils/binutils
Message-ID:  <201209071803.q87I3TJ8040766@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: dim
Date: Fri Sep  7 18:03:29 2012
New Revision: 240208
URL: http://svn.freebsd.org/changeset/base/240208

Log:
  MFC r239962:
  
  Fix a twelve year old bug in readelf: when process_dynamic_segment()
  encounters a DT_RUNPATH entry, the global dynamic_info[] array is
  overrun, causing some other global variable to be overwritten.
  
  In my testcase, this was the section_headers variable, leading to
  segfaults or jemalloc assertions when it was freed later on.
  
  Thanks to Koop Mast for providing samples of a few "bad" .so files.

Modified:
  stable/9/contrib/binutils/binutils/readelf.c
Directory Properties:
  stable/9/contrib/binutils/   (props changed)

Changes in other areas also in this revision:
Modified:
  stable/7/contrib/binutils/binutils/readelf.c
  stable/8/contrib/binutils/binutils/readelf.c
Directory Properties:
  stable/7/contrib/binutils/   (props changed)
  stable/8/contrib/binutils/   (props changed)

Modified: stable/9/contrib/binutils/binutils/readelf.c
==============================================================================
--- stable/9/contrib/binutils/binutils/readelf.c	Fri Sep  7 17:58:36 2012	(r240207)
+++ stable/9/contrib/binutils/binutils/readelf.c	Fri Sep  7 18:03:29 2012	(r240208)
@@ -174,7 +174,7 @@ static Elf_Internal_Syminfo *dynamic_sym
 static unsigned long dynamic_syminfo_offset;
 static unsigned int dynamic_syminfo_nent;
 static char program_interpreter[PATH_MAX];
-static bfd_vma dynamic_info[DT_JMPREL + 1];
+static bfd_vma dynamic_info[DT_ENCODING];
 static bfd_vma dynamic_info_DT_GNU_HASH;
 static bfd_vma version_info[16];
 static Elf_Internal_Ehdr elf_header;



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201209071803.q87I3TJ8040766>