Hi,

On Tue, Nov 06, 2012 at 11:03:36AM +0100, 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.

ACK (having read the through the relevant bits of init.c and through
pf_init_context()).

gert
-- 
USENET is *not* the non-clickable part of WWW!
                                                           //www.muc.de/~gert/
Gert Doering - Munich, Germany                             g...@greenie.muc.de
fax: +49-89-35655025                        g...@net.informatik.tu-muenchen.de

Attachment: pgpAQ3y0IZLVm.pgp
Description: PGP signature

Reply via email to