Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 01 Sep 2012 13:06:51 +0200
From:      Dimitry Andric <dim@FreeBSD.org>
To:        Garrett Cooper <yanegomi@gmail.com>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r239962 - head/contrib/binutils/binutils
Message-ID:  <5041EC4B.5000501@FreeBSD.org>
In-Reply-To: <CAGH67wS=5-oWRWwBS9UVg4N6GCmmx3kVJku6Cy6NZmEbF1W3Pg@mail.gmail.com>
References:  <201208312328.q7VNSfLK012047@svn.freebsd.org> <CAGH67wS=5-oWRWwBS9UVg4N6GCmmx3kVJku6Cy6NZmEbF1W3Pg@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On 2012-09-01 03:00, Garrett Cooper wrote:
> On Fri, Aug 31, 2012 at 4:28 PM, Dimitry Andric <dim@freebsd.org> wrote:
>> Author: dim
>> Date: Fri Aug 31 23:28:41 2012
>> New Revision: 239962
>> URL: http://svn.freebsd.org/changeset/base/239962
>>
>> Log:
>>    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.
...
>      Nice! Hopefully this fixes the random tinderbox crashes I've seen!

Koop said it occurred semi-randomly during the openjdk7 port build.  It
apparently runs "readelf -d" on a whole bunch of produced libraries.  So
I hope those are now gone. :)


> Has this been fixed upstream?

Yes, in 2009:

http://sourceware.org/git/?p=binutils.git;a=commitdiff;h=839d59b5c25836c05cc5abb401e0b5fe17a98981



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