From: Tobias Brunner <tobias@strongswan.org>
Date: Fri, 28 Sep 2012 13:13:17 +0000 (+0200)
Subject: Properly cleanup varargs in LDAP fetcher's set_option()
X-Git-Tag: 5.0.1~35
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=9fa335cb1b8fd92a60b5a1ae5f25a03b9d3ec030;p=thirdparty%2Fstrongswan.git

Properly cleanup varargs in LDAP fetcher's set_option()
---

diff --git a/src/libstrongswan/plugins/ldap/ldap_fetcher.c b/src/libstrongswan/plugins/ldap/ldap_fetcher.c
index fc6114b0af..75f9648532 100644
--- a/src/libstrongswan/plugins/ldap/ldap_fetcher.c
+++ b/src/libstrongswan/plugins/ldap/ldap_fetcher.c
@@ -176,13 +176,14 @@ METHOD(fetcher_t, set_option, bool,
 	switch (option)
 	{
 		case FETCH_TIMEOUT:
-		{
 			this->timeout = va_arg(args, u_int);
-			return TRUE;
-		}
+			break;
 		default:
+			va_end(args);
 			return FALSE;
 	}
+	va_end(args);
+	return TRUE;
 }
 
 METHOD(fetcher_t, destroy, void,