commit 9c9f1c1a23609d4c1db4459937716b5b305c2cdc
author Himanshu Agarwal <himanaga@codeaurora.org> Mon Feb 12 19:38:09 2018 +0530
committer Gerrit - the friendly Code Review server <code-review@localhost> Mon Mar 12 23:06:42 2018 -0700
tree ea69708bfec95cf058503d353095667d21302fb0
parent 3c23c93c00f21c5f4fe1cc7c1fc2601cae67fa34

wlan: Free assoc responses while doing lim cleanup

In case when tx confirmation for assoc response is not coming
from fw and driver unload happens, all the assoc responses from
the list should be freed while doing lim cleanup. But in
limDestroyAssocRspCompletiontionList(), assoc responses in the
list does not get freed causing memory leak.

Free assoc responses while doing lim cleanup in
limDestroyAssocRspCompletiontionList().

Change-Id: I476a044dcd84b08c6c03ea11ada60e88383e4048
CRs-Fixed: 2188215

CORE/MAC/src/pe/lim/limApi.c

diff --git a/CORE/MAC/src/pe/lim/limApi.c b/CORE/MAC/src/pe/lim/limApi.c
index d8137ff..988eec3 100644
--- a/CORE/MAC/src/pe/lim/limApi.c
+++ b/CORE/MAC/src/pe/lim/limApi.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2011-2017 The Linux Foundation. All rights reserved.
+ * Copyright (c) 2011-2018 The Linux Foundation. All rights reserved.
  *
  * Previously licensed under the ISC license by Qualcomm Atheros, Inc.
  *
@@ -712,6 +712,16 @@
 static void
 limDestroyAssocRspCompletiontionList(tpAniSirGlobal pMac)
 {
+    assoc_rsp_tx_context *pAssocRspCtx;
+
+    while(vos_list_remove_front(&pMac->assoc_rsp_completion_list,
+           (vos_list_node_t**)&pAssocRspCtx) == VOS_STATUS_SUCCESS)
+    {
+        VOS_TRACE(VOS_MODULE_ID_PE, VOS_TRACE_LEVEL_INFO,
+                  FL("Fixing leak! Deallocating pAssocRspCtx node %lu"),
+                      (unsigned long)pAssocRspCtx);
+        vos_mem_free(pAssocRspCtx);
+    }
     vos_list_destroy(&pMac->assoc_rsp_completion_list);
 }