From e33510f8a32ffe72f6c239919058e6eb120ef070 Mon Sep 17 00:00:00 2001 From: Martin Willi Date: Tue, 13 Dec 2011 11:39:54 +0100 Subject: [PATCH] Fixed leak of shared keys in xauth-generic plugin --- src/libcharon/plugins/xauth_generic/xauth_generic.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/libcharon/plugins/xauth_generic/xauth_generic.c b/src/libcharon/plugins/xauth_generic/xauth_generic.c index cac740a34..981ab77d0 100644 --- a/src/libcharon/plugins/xauth_generic/xauth_generic.c +++ b/src/libcharon/plugins/xauth_generic/xauth_generic.c @@ -73,6 +73,7 @@ METHOD(xauth_method_t, process_peer, status_t, CONFIGURATION_ATTRIBUTE_V1, XAUTH_USER_NAME, user)); cp->add_attribute(cp, configuration_attribute_create_chunk( CONFIGURATION_ATTRIBUTE_V1, XAUTH_USER_PASSWORD, pass)); + shared->destroy(shared); *out = cp; return NEED_MORE; } @@ -151,6 +152,7 @@ METHOD(xauth_method_t, process_server, status_t, { DBG2(DBG_IKE, "authentication of '%Y' with XAuth successful", peer); } + DESTROY_IF(shared); DESTROY_IF(id); return status; }