Re: [Openvpn-devel] [PATCH] Fix double-free issue in pf_destroy_context()

2012-11-06 Thread David Sommerseth
On 06/11/12 11:03, David Sommerseth wrote: > From: David Sommerseth > > 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_

Re: [Openvpn-devel] [PATCH] Fix double-free issue in pf_destroy_context()

2012-11-06 Thread Gert Doering
Hi, On Tue, Nov 06, 2012 at 11:03:36AM +0100, David Sommerseth wrote: > From: David Sommerseth > > 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 considera

[Openvpn-devel] [PATCH] Fix double-free issue in pf_destroy_context()

2012-11-06 Thread David Sommerseth
From: David Sommerseth 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_al