[PATCH] fix memory leak in Juniper logout function (caught with valgrind)

Daniel Lenski dlenski at gmail.com
Sun Apr 15 16:50:04 PDT 2018


Signed-off-by: Daniel Lenski <dlenski at gmail.com>
---
 oncp.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/oncp.c b/oncp.c
index dbfb1ef..c5fd34a 100644
--- a/oncp.c
+++ b/oncp.c
@@ -1297,6 +1297,7 @@ int oncp_bye(struct openconnect_info *vpninfo, const char *reason)
 	orig_path = vpninfo->urlpath;
 	vpninfo->urlpath = strdup("dana-na/auth/logout.cgi"); /* redirect segfaults without strdup */
 	ret = do_https_request(vpninfo, "GET", NULL, NULL, &res_buf, 0);
+	free(vpninfo->urlpath);
 	vpninfo->urlpath = orig_path;
 
 	if (ret < 0)
-- 
2.7.4




More information about the openconnect-devel mailing list