diff options
| author | Paul Stewart <pstew@chromium.org> | 2012-08-04 20:10:08 +0300 |
|---|---|---|
| committer | Arik Nemtsov <arik@wizery.com> | 2012-08-08 23:48:03 +0300 |
| commit | f2f1caae16829f02fd000188bd696d5247f97442 (patch) | |
| tree | 9dcd1d833941b0f563a2ca2f78d3163cef1bd358 | |
| parent | bac82c9a17973f97597dddef9f3840b350ca491c (diff) | |
| download | external_wpa_supplicant_8_ti-f2f1caae16829f02fd000188bd696d5247f97442.zip external_wpa_supplicant_8_ti-f2f1caae16829f02fd000188bd696d5247f97442.tar.gz external_wpa_supplicant_8_ti-f2f1caae16829f02fd000188bd696d5247f97442.tar.bz2 | |
Fix a couple memory leaks
Found using valgrind.
Signed-hostap: Paul Stewart <pstew@chromium.org>
| -rw-r--r-- | src/drivers/driver_nl80211.c | 5 | ||||
| -rw-r--r-- | wpa_supplicant/dbus/dbus_new_handlers.c | 1 |
2 files changed, 5 insertions, 1 deletions
diff --git a/src/drivers/driver_nl80211.c b/src/drivers/driver_nl80211.c index a0145db..f7b56b2 100644 --- a/src/drivers/driver_nl80211.c +++ b/src/drivers/driver_nl80211.c @@ -8602,12 +8602,15 @@ static int nl80211_signal_monitor(void *priv, int threshold, int hysteresis) cqm = nlmsg_alloc(); if (cqm == NULL) - return -1; + goto nla_put_failure; NLA_PUT_U32(cqm, NL80211_ATTR_CQM_RSSI_THOLD, threshold); NLA_PUT_U32(cqm, NL80211_ATTR_CQM_RSSI_HYST, hysteresis); nla_put_nested(msg, NL80211_ATTR_CQM, cqm); + nlmsg_free(cqm); + cqm = NULL; + if (send_and_recv_msgs(drv, msg, NULL, NULL) == 0) return 0; msg = NULL; diff --git a/wpa_supplicant/dbus/dbus_new_handlers.c b/wpa_supplicant/dbus/dbus_new_handlers.c index 8145a70..3f895cf 100644 --- a/wpa_supplicant/dbus/dbus_new_handlers.c +++ b/wpa_supplicant/dbus/dbus_new_handlers.c @@ -609,6 +609,7 @@ DBusMessage * wpas_dbus_handler_create_interface(DBusMessage *message, out: os_free(driver); os_free(ifname); + os_free(confname); os_free(bridge_ifname); return reply; |