diff options
| author | Sujith Manoharan <c_manoha@qca.qualcomm.com> | 2013-01-09 16:07:48 +0530 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-02-03 18:24:41 -0600 |
| commit | 657273897cd2f9b7964ac704c6e58fbe8295c64d (patch) | |
| tree | 85e6bbe768ad4a0750c35727394baab00849f009 /drivers/net/wireless/ath | |
| parent | e4fcbe5297a8c6e512c68a77c660a8cdd7d315e4 (diff) | |
| download | linux-3.10-657273897cd2f9b7964ac704c6e58fbe8295c64d.tar.gz linux-3.10-657273897cd2f9b7964ac704c6e58fbe8295c64d.tar.bz2 linux-3.10-657273897cd2f9b7964ac704c6e58fbe8295c64d.zip | |
ath9k_htc: Fix memory leak
commit 0981c3b24ef664f5611008a6e6d0622fac6d892b upstream.
SKBs that are allocated in the HTC layer do not have callbacks
registered and hence ended up not being freed, Fix this by freeing
them properly in the TX completion routine.
Reported-by: Larry Finger <Larry.Finger@lwfinger.net>
Signed-off-by: Sujith Manoharan <c_manoha@qca.qualcomm.com>
Tested-by: Larry Finger <Larry.Finger@lwfinger.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/net/wireless/ath')
| -rw-r--r-- | drivers/net/wireless/ath/ath9k/htc_hst.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/net/wireless/ath/ath9k/htc_hst.c b/drivers/net/wireless/ath/ath9k/htc_hst.c index c25226a32dd..ec86d981abc 100644 --- a/drivers/net/wireless/ath/ath9k/htc_hst.c +++ b/drivers/net/wireless/ath/ath9k/htc_hst.c @@ -342,6 +342,8 @@ void ath9k_htc_txcompletion_cb(struct htc_target *htc_handle, endpoint->ep_callbacks.tx(endpoint->ep_callbacks.priv, skb, htc_hdr->endpoint_id, txok); + } else { + kfree_skb(skb); } } |