diff options
| author | Vignesh Viswanathan <viswanat@codeaurora.org> | 2018-10-16 17:09:14 +0530 |
|---|---|---|
| committer | nshrivas <nshrivas@codeaurora.org> | 2018-11-05 17:20:35 -0800 |
| commit | b38a232b23330bdc7ed0a2fcbca9e91f8eeeb77c (patch) | |
| tree | 0fd5a4340eca119c1725e86a60dcd594eaf18edc /core | |
| parent | a633e2d4bda3c77441fc520135c92e8a8fc06cfb (diff) | |
qcacld-3.0: Free sta_self_wmi_rsp if WMA_DELETE_STA_REQ is not queued
In wma_self_peer_remove, sta_self_wmi_rsp is allocated and sent as
params to wma_fill_hold_req for WMA_DELETE_STA_REQ. However, if
wma_fill_hold_req fails and returns NULL, the memory allocated for
sta_self_wmi_rsp is not freed leading to memory leak.
Free sta_self_wmi_rsp in the error handling case for wma_fill_hold_req
in wma_self_peer_remove.
Change-Id: I8df840b39284e00a8c8be114c0fec13c3c199fc6
CRs-Fixed: 2334580
Diffstat (limited to 'core')
| -rw-r--r-- | core/wma/src/wma_dev_if.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/core/wma/src/wma_dev_if.c b/core/wma/src/wma_dev_if.c index 5a1d90c515f6..847a4e9dc668 100644 --- a/core/wma/src/wma_dev_if.c +++ b/core/wma/src/wma_dev_if.c @@ -532,6 +532,7 @@ static QDF_STATUS wma_self_peer_remove(tp_wma_handle wma_handle, vdev_id); wma_remove_req(wma_handle, vdev_id, WMA_DEL_P2P_SELF_STA_RSP_START); + qdf_mem_free(sta_self_wmi_rsp); qdf_status = QDF_STATUS_E_FAILURE; goto error; } |