Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 4 Jul 2016 23:15:31 -0400
From:      Julio Merino <julio@meroh.net>
To:        "Conrad E. Meyer" <cem@freebsd.org>
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org,  svn-src-head@freebsd.org
Subject:   Re: svn commit: r299496 - head/contrib/atf/atf-c/detail
Message-ID:  <CADyfeQVL5tmRyjW478%2BSiAO_kMY9brrVeMs5Jus2kfi1Egv3ww@mail.gmail.com>
In-Reply-To: <201605112339.u4BNdehd062987@repo.freebsd.org>
References:  <201605112339.u4BNdehd062987@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, May 11, 2016 at 7:39 PM, Conrad E. Meyer <cem@freebsd.org> wrote:
> Author: cem
> Date: Wed May 11 23:39:39 2016
> New Revision: 299496
> URL: https://svnweb.freebsd.org/changeset/base/299496
>
> Log:
>   atf map: Fix double-free in low memory error path
>
>   If atf_list_append(, X, ) fails, X is freed.  Don't free it again.

This fix seems bogus. I think the following is correct:

https://github.com/jmmv/atf/commit/710d1cf96f3ab716e76b569714fd7303292d93b2

-- 
Julio Merino -- http://julio.meroh.net/



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CADyfeQVL5tmRyjW478%2BSiAO_kMY9brrVeMs5Jus2kfi1Egv3ww>