From fe6b002f0f06cefa801b1d0a8603b95b52d27c62 Mon Sep 17 00:00:00 2001 From: Thomas Haller Date: Tue, 19 Aug 2014 00:21:54 +0200 Subject: [PATCH] settings: fix memory leak in get_settings_auth_cb() Signed-off-by: Thomas Haller --- src/settings/nm-settings-connection.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/settings/nm-settings-connection.c b/src/settings/nm-settings-connection.c index e8d5af0399..8c9ea81247 100644 --- a/src/settings/nm-settings-connection.c +++ b/src/settings/nm-settings-connection.c @@ -1162,10 +1162,9 @@ get_settings_auth_cb (NMSettingsConnection *self, */ bssid_list = nm_settings_connection_get_seen_bssids (self); s_wifi = nm_connection_get_setting_wireless (NM_CONNECTION (dupl_con)); - if (bssid_list && s_wifi) { + if (bssid_list && s_wifi) g_object_set (s_wifi, NM_SETTING_WIRELESS_SEEN_BSSIDS, bssid_list, NULL); - g_slist_free_full (bssid_list, g_free); - } + g_slist_free_full (bssid_list, g_free); /* 802-11-wireless.security property is deprecated. But we set it here so that * we don't disturb old clients that might expect it being properly set for