commit 5d1c8ad11de48ba5e449e83e1e5dcd844e9c35d1
author Jouni Malinen <j@w1.fi> Tue Apr 23 12:34:56 2013 -0700
committer Dmitry Shmidt <dimitrysh@google.com> Tue Apr 23 12:36:11 2013 -0700
tree ad9b128d888d3241f5e3b50d7534cfcee2081b6d
parent 97d980693a6550ea7a66d85cf1f453b1246436b2

wpa_supplicant: fix memory leak in -I option handling

Change-Id: I1b393d245d606882e7bb548135d1eab99a988a81
Signed-off-by: Dmitry Shmidt <dimitrysh@google.com>

wpa_supplicant/config.h

diff --git a/wpa_supplicant/config.h b/wpa_supplicant/config.h
index f83499c..4a175ce 100644
--- a/wpa_supplicant/config.h
+++ b/wpa_supplicant/config.h
@@ -912,7 +912,7 @@
  * wpa_config_read - Read and parse configuration database
  * @name: Name of the configuration (e.g., path and file name for the
  * configuration file)
- * @cfgp: Pointer to previousely allocated configuration data
+ * @cfgp: Pointer to previously allocated configuration data or %NULL if none
  * Returns: Pointer to allocated configuration data or %NULL on failure
  *
  * This function reads configuration data, parses its contents, and allocates

wpa_supplicant/main.c

diff --git a/wpa_supplicant/main.c b/wpa_supplicant/main.c
index c3ecbb3..1744620 100644
--- a/wpa_supplicant/main.c
+++ b/wpa_supplicant/main.c
@@ -32,7 +32,8 @@
 	       "        [-o<override driver>] [-O<override ctrl>] \\\n"
 	       "        [-N -i<ifname> -c<conf> [-C<ctrl>] "
 	       "[-D<driver>] \\\n"
-	       "        [-p<driver_param>] [-b<br_ifname>] [-I<config file>]\n"
+	       "        [-p<driver_param>] [-b<br_ifname>] [-I<config file>] "
+	       "...]\n"
 	       "\n"
 	       "drivers:\n",
 	       wpa_supplicant_version, wpa_supplicant_license);
@@ -50,7 +51,7 @@
 	       "  -c = Configuration file\n"
 	       "  -C = ctrl_interface parameter (only used if -c is not)\n"
 	       "  -i = interface name\n"
-	       "  -I = additional Configuration file\n"
+	       "  -I = additional configuration file\n"
 	       "  -d = increase debugging verbosity (-dd even more)\n"
 	       "  -D = driver name (can be multiple drivers: nl80211,wext)\n"
 	       "  -e = entropy file\n");

wpa_supplicant/wpa_supplicant.c

diff --git a/wpa_supplicant/wpa_supplicant.c b/wpa_supplicant/wpa_supplicant.c
index 398fca1..4483173 100644
--- a/wpa_supplicant/wpa_supplicant.c
+++ b/wpa_supplicant/wpa_supplicant.c
@@ -399,6 +399,9 @@
 	os_free(wpa_s->confname);
 	wpa_s->confname = NULL;
 
+	os_free(wpa_s->confanother);
+	wpa_s->confanother = NULL;
+
 	wpa_sm_set_eapol(wpa_s->wpa, NULL);
 	eapol_sm_deinit(wpa_s->eapol);
 	wpa_s->eapol = NULL;

wpa_supplicant/wpa_supplicant_i.h

diff --git a/wpa_supplicant/wpa_supplicant_i.h b/wpa_supplicant/wpa_supplicant_i.h
index b47c11b..f96b245 100644
--- a/wpa_supplicant/wpa_supplicant_i.h
+++ b/wpa_supplicant/wpa_supplicant_i.h
@@ -56,9 +56,10 @@
 	const char *confname;
 
 	/**
-	 * confanother - Additional Configuration name (file or profile) name
+	 * confanother - Additional configuration name (file or profile) name
 	 *
-	 * This can also be %NULL when a configuration file is not used.
+	 * This can also be %NULL when the additional configuration file is not
+	 * used.
 	 */
 	const char *confanother;