diff options
| author | yeshwanth sriram guntuka <ysriramg@codeaurora.org> | 2017-04-26 17:12:23 +0530 |
|---|---|---|
| committer | snandini <snandini@codeaurora.org> | 2017-04-28 18:06:42 -0700 |
| commit | 1d38201150aeed7b18e4048903a5e86d5aafe0a9 (patch) | |
| tree | fac97c2a5e1ed4ea476147d665a9814598448df1 /core/wma/src/wma_mgmt.c | |
| parent | 94df5e2a8a5ce508fb0cde505dcc00f94976eff4 (diff) | |
| download | qcacld-1d38201150aeed7b18e4048903a5e86d5aafe0a9.tar.gz | |
qcacld-3.0: Fix memory leaks in multiple functions
1) Fix Wma target req msg,freed twice in wma_cleanup_vdev_resp_queue.
2) Packet buffer not freed if mgmt tx completion
is not received from firmware before driver unload.
Change-Id: I2a7e5f8dc0993588b569093d64cfba293069ae23
CRs-Fixed: 2038479
Diffstat (limited to 'core/wma/src/wma_mgmt.c')
| -rw-r--r-- | core/wma/src/wma_mgmt.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/core/wma/src/wma_mgmt.c b/core/wma/src/wma_mgmt.c index a9e34a97cc..d8734b7320 100644 --- a/core/wma/src/wma_mgmt.c +++ b/core/wma/src/wma_mgmt.c @@ -2628,9 +2628,11 @@ static int wma_process_mgmt_tx_completion(tp_wma_handle wma_handle, packetdump_cb(wmi_desc->nbuf, QDF_STATUS_SUCCESS, wmi_desc->vdev_id, TX_MGMT_PKT); - if (wmi_desc->tx_cmpl_cb) + if (wmi_desc->tx_cmpl_cb) { wmi_desc->tx_cmpl_cb(wma_handle->mac_context, wmi_desc->nbuf, 1); + wmi_desc->nbuf = NULL; + } if (wmi_desc->ota_post_proc_cb) wmi_desc->ota_post_proc_cb((tpAniSirGlobal) |