diff options
| author | Alfonso Sánchez-Beato <alfonso.sanchez-beato@canonical.com> | 2018-05-28 16:25:36 +0200 |
|---|---|---|
| committer | Lubomir Rintel <lkundrak@v3.sk> | 2018-05-31 11:53:26 +0200 |
| commit | cdbd99c5d5b5a62c5d4bf4d5e7a49e97e307d987 (patch) | |
| tree | d015e460cdf9c4468517be5ce2a0b93c1e1a613d | |
| parent | 581ce347fbcff0f8190ad200f6314592a209d818 (diff) | |
| download | NetworkManager-cdbd99c5d5b5a62c5d4bf4d5e7a49e97e307d987.tar.gz | |
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.
| -rw-r--r-- | src/platform/wifi/wifi-utils-nl80211.c | 8 |
1 files 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; } |