diff options
author | Michal Kazior <michal.kazior@tieto.com> | 2013-08-13 01:59:38 -0400 |
---|---|---|
committer | Kalle Valo <kvalo@qca.qualcomm.com> | 2013-08-14 11:01:28 -0400 |
commit | 7f8a62db4ba9d08d2bacf126b442d43ea47b22a8 (patch) | |
tree | e290cb5a1ad7ec05474979e9c0291736c9549db6 /drivers/net | |
parent | dfa95b5024027400207ed6bf8914936f1b4efa88 (diff) |
ath10k: plug possible memory leak in WMI
There was a possible memory leak when WMI command
queue reached it's limit. Command buffers were not
freed.
Signed-off-by: Michal Kazior <michal.kazior@tieto.com>
Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
Diffstat (limited to 'drivers/net')
-rw-r--r-- | drivers/net/wireless/ath/ath10k/wmi.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/net/wireless/ath/ath10k/wmi.c b/drivers/net/wireless/ath/ath10k/wmi.c index 55f90c761868..775fedfd6832 100644 --- a/drivers/net/wireless/ath/ath10k/wmi.c +++ b/drivers/net/wireless/ath/ath10k/wmi.c | |||
@@ -110,6 +110,7 @@ static int ath10k_wmi_cmd_send(struct ath10k *ar, struct sk_buff *skb, | |||
110 | if (atomic_add_return(1, &ar->wmi.pending_tx_count) > | 110 | if (atomic_add_return(1, &ar->wmi.pending_tx_count) > |
111 | WMI_MAX_PENDING_TX_COUNT) { | 111 | WMI_MAX_PENDING_TX_COUNT) { |
112 | /* avoid using up memory when FW hangs */ | 112 | /* avoid using up memory when FW hangs */ |
113 | dev_kfree_skb(skb); | ||
113 | atomic_dec(&ar->wmi.pending_tx_count); | 114 | atomic_dec(&ar->wmi.pending_tx_count); |
114 | return -EBUSY; | 115 | return -EBUSY; |
115 | } | 116 | } |