diff options
| author | Vignesh Viswanathan <viswanat@codeaurora.org> | 2018-01-17 19:44:13 +0530 |
|---|---|---|
| committer | snandini <snandini@codeaurora.org> | 2018-01-18 00:36:59 -0800 |
| commit | c48c066f6ed77620a6175fcb34a145ebc4266066 (patch) | |
| tree | aa9413264c07378dc5f2539d7e33fa9ff8ebb892 | |
| parent | 105360628a3b91997311b49679db6cf283a8a4cb (diff) | |
qcacld-3.0: Free IE buffers in SAP config in case of SSR SAP Restart
When a SSR SAP restart happens, hdd_ssr_restart_sap invokes
wlan_hdd_start_sap. In this function, IE buffers for sap_config is
allocated as part of wlan_hdd_cfg80211_update_apies and then copied
to csr_profile in wlansap_start_bss and then SAP FSM Starts the SAP.
The memory allocated for IE buffers in sap_config needs to be freed
in both SAP Start Success and failure cases without which, in case
of multiple SSRs, each time new malloc is done and will lead to
memory leak.
Free the IE buffers in sap_config in wlan_hdd_start_sap in both
SAP Start success and failure cases.
Change-Id: I43035c786444d0dfbcfaba9038a09cf345263b10
CRs-Fixed: 2168107
| -rw-r--r-- | core/hdd/src/wlan_hdd_main.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/core/hdd/src/wlan_hdd_main.c b/core/hdd/src/wlan_hdd_main.c index 04663596d582..092f47256356 100644 --- a/core/hdd/src/wlan_hdd_main.c +++ b/core/hdd/src/wlan_hdd_main.c @@ -11634,7 +11634,6 @@ void wlan_hdd_start_sap(hdd_adapter_t *ap_adapter, bool reinit) if (0 != wlan_hdd_cfg80211_update_apies(ap_adapter)) { hdd_err("SAP Not able to set AP IEs"); - wlansap_reset_sap_config_add_ie(sap_config, eUPDATE_IE_ALL); goto end; } @@ -11652,6 +11651,7 @@ void wlan_hdd_start_sap(hdd_adapter_t *ap_adapter, bool reinit) hdd_err("SAP Start failed"); goto end; } + wlansap_reset_sap_config_add_ie(sap_config, eUPDATE_IE_ALL); hdd_info("SAP Start Success"); set_bit(SOFTAP_BSS_STARTED, &ap_adapter->event_flags); if (hostapd_state->bssState == BSS_START) @@ -11663,6 +11663,7 @@ void wlan_hdd_start_sap(hdd_adapter_t *ap_adapter, bool reinit) return; end: + wlansap_reset_sap_config_add_ie(sap_config, eUPDATE_IE_ALL); mutex_unlock(&hdd_ctx->sap_lock); /* SAP context and beacon cleanup will happen during driver unload * in hdd_stop_adapter |