ath9k_htc: Fix HTC layer memleak
authorSujith <Sujith.Manoharan@atheros.com>
Tue, 6 Apr 2010 09:58:19 +0000 (15:28 +0530)
committerJohn W. Linville <linville@tuxdriver.com>
Thu, 8 Apr 2010 19:24:11 +0000 (15:24 -0400)
Messages that are generated by the HTC layer
don't have any TX callback endpoints assigned to them.
Consequently, the allocated SKBs are never freed.

Fix this issue by handling this case in the HTC layer
itself.

Signed-off-by: Sujith <Sujith.Manoharan@atheros.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/ath/ath9k/htc_hst.c

index 30f608bfc5671c3f1f9c34e6cc3b7bc2a69a667d..24d7b886fe233a8095ec544cc9358c51c3c43913 100644 (file)
@@ -326,11 +326,13 @@ void ath9k_htc_txcompletion_cb(struct htc_target *htc_handle,
        if (htc_handle->htc_flags & HTC_OP_CONFIG_PIPE_CREDITS) {
                complete(&htc_handle->cmd_wait);
                htc_handle->htc_flags &= ~HTC_OP_CONFIG_PIPE_CREDITS;
+               goto ret;
        }
 
        if (htc_handle->htc_flags & HTC_OP_START_WAIT) {
                complete(&htc_handle->cmd_wait);
                htc_handle->htc_flags &= ~HTC_OP_START_WAIT;
+               goto ret;
        }
 
        if (skb) {
@@ -343,6 +345,11 @@ void ath9k_htc_txcompletion_cb(struct htc_target *htc_handle,
                                                  htc_hdr->endpoint_id, txok);
                }
        }
+
+       return;
+ret:
+       /* HTC-generated packets are freed here. */
+       dev_kfree_skb_any(skb);
 }
 
 /*