From cdbd99c5d5b5a62c5d4bf4d5e7a49e97e307d987 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Alfonso=20S=C3=A1nchez-Beato?= Date: Mon, 28 May 2018 16:25:36 +0200 Subject: [PATCH] platform/wifi: do not double-free nl_msg In some places, there was an unneeded call to nlmsg_free () for messages declared with the nm_auto_nlmsg macro. --- src/platform/wifi/wifi-utils-nl80211.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/src/platform/wifi/wifi-utils-nl80211.c b/src/platform/wifi/wifi-utils-nl80211.c index db187a1fb3..767e23a91d 100644 --- a/src/platform/wifi/wifi-utils-nl80211.c +++ b/src/platform/wifi/wifi-utils-nl80211.c @@ -250,8 +250,7 @@ wifi_nl80211_set_mode (WifiData *data, const NM80211Mode mode) err = nl80211_send_and_recv (nl80211, msg, NULL, NULL); return err >= 0; - nla_put_failure: - nlmsg_free (msg); +nla_put_failure: return FALSE; } @@ -269,7 +268,6 @@ wifi_nl80211_set_powersave (WifiData *data, guint32 powersave) return err >= 0; nla_put_failure: - nlmsg_free (msg); return FALSE; } @@ -542,8 +540,7 @@ nl80211_get_ap_info (WifiDataNl80211 *nl80211, return; - nla_put_failure: - nlmsg_free (msg); +nla_put_failure: return; } @@ -597,7 +594,6 @@ wifi_nl80211_indicate_addressing_running (WifiData *data, gboolean running) return err >= 0; nla_put_failure: - nlmsg_free (msg); return FALSE; }