Free CSTP option structure before error return if malloc fails

Spotted by Coverity. Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
sailfishos-mirror · Sep 26, 2012 · 7e952ad · 7e952ad
1 parent bda1490
commit 7e952ad
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/cstp.c b/cstp.c
@@ -302,6 +302,9 @@ static int start_cstp_connection(struct openconnect_info *vpninfo)
 
 		if (!new_option->option || !new_option->value) {
 			vpn_progress(vpninfo, PRG_ERR, _("No memory for options\n"));
+			free(new_option->option);
+			free(new_option->value);
+			free(new_option);
 			return -ENOMEM;
 		}