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
pgpAQ3y0IZLVm.pgp
Description: PGP signature