diff options
author | Sara Sharon <sara.sharon@intel.com> | 2018-02-19 07:48:35 -0500 |
---|---|---|
committer | Johannes Berg <johannes.berg@intel.com> | 2018-02-19 08:53:08 -0500 |
commit | d78d9ee9d40aca4781d2c5334972544601a4c3a2 (patch) | |
tree | fe02c96847d7caef8d07a4cd5b553250d853feb5 /net | |
parent | 651b9920d7a694ffb1f885aef2bbb068a25d9d66 (diff) |
mac80211: fix a possible leak of station stats
If sta_info_alloc fails after allocating the per CPU statistics,
they are not properly freed.
Fixes: c9c5962b56c1 ("mac80211: enable collecting station statistics per-CPU")
Signed-off-by: Sara Sharon <sara.sharon@intel.com>
Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'net')
-rw-r--r-- | net/mac80211/sta_info.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/net/mac80211/sta_info.c b/net/mac80211/sta_info.c index 0c5627f8a104..8d7e3732bb61 100644 --- a/net/mac80211/sta_info.c +++ b/net/mac80211/sta_info.c | |||
@@ -433,6 +433,7 @@ free_txq: | |||
433 | if (sta->sta.txq[0]) | 433 | if (sta->sta.txq[0]) |
434 | kfree(to_txq_info(sta->sta.txq[0])); | 434 | kfree(to_txq_info(sta->sta.txq[0])); |
435 | free: | 435 | free: |
436 | free_percpu(sta->pcpu_rx_stats); | ||
436 | #ifdef CONFIG_MAC80211_MESH | 437 | #ifdef CONFIG_MAC80211_MESH |
437 | kfree(sta->mesh); | 438 | kfree(sta->mesh); |
438 | #endif | 439 | #endif |