On 06/11/12 11:03, David Sommerseth wrote: > From: David Sommerseth <dav...@redhat.com> > > In commit dc7be6d078ba106f9b0de12f3e879c3561c3c537 the string_alloc() call > in pf_init_context() was modified to use the gc_arena object for memory > allocation. What was not taken into consideration was that > pf_destroy_context() > was also freeing memory allocated by string_alloc(), and when > pf_init_context() > is calling gc_free() a double-free situation showed up. > > Lets remove the explict free, and let gc_free take care of all the memory > handling. > > Reported-by: c...@163.com > Signed-off-by: David Sommerseth <dav...@redhat.com> > --- > src/openvpn/pf.c | 1 - > 1 file changed, 1 deletion(-)
Applied to master and beta/2.3 branches.
commit 1f300fe94f1bd521966bb05dea40edc1fae82b84 (master)
commit 03dfcd95b29eda81aee117bf9d28c001f2521e15 (beta/2.3)
Author: David Sommerseth <dav...@redhat.com>
List-Post: openvpn-devel@lists.sourceforge.net
Date: Tue Nov 6 10:59:01 2012 +0100
Fix double-free issue in pf_destroy_context()
Reported-by: c...@163.com
Signed-off-by: David Sommerseth <dav...@redhat.com>
Acked-by: Gert Doering <g...@greenie.muc.de>
Message-Id: 1352196216-11560-1-git-send-email-d...@users.sourceforge.net
URL: http://article.gmane.org/gmane.network.openvpn.devel/7124
kind regards,
David Sommerseth
signature.asc
Description: OpenPGP digital signature
