From f2f1caae16829f02fd000188bd696d5247f97442 Mon Sep 17 00:00:00 2001 From: Paul Stewart Date: Sat, 4 Aug 2012 20:10:08 +0300 Subject: Fix a couple memory leaks Found using valgrind. Signed-hostap: Paul Stewart --- src/drivers/driver_nl80211.c | 5 ++++- wpa_supplicant/dbus/dbus_new_handlers.c | 1 + 2 files changed, 5 insertions(+), 1 deletion(-) 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; -- cgit v1.1