From 465429062f8ba7b8eaec6fd8519616e8bdfe655c Mon Sep 17 00:00:00 2001 From: Yeshwanth Sriram Guntuka Date: Tue, 17 Oct 2017 17:55:45 +0530 Subject: qcacld-3.0: Check sme session opened flag for opened interfaces Sap context memory is allocated on adding softap inteface. If wlan interface is disabled after softap interface is added, iface_idle_work is scheduled since DEVICE_IFACE_OPENED will not be set. Memory leak will happen as sap context is not freed in this case. Fix is to check for DEVICE_IFACE_OPENED or SME_SESSION_OPENED flag for opened interfaces. Change-Id: I3585078135dc042222a92cf5c896ad26a086e506 CRs-Fixed: 2126666 --- core/hdd/src/wlan_hdd_main.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/core/hdd/src/wlan_hdd_main.c b/core/hdd/src/wlan_hdd_main.c index 73eb8c01dd31..0587a0e785ef 100644 --- a/core/hdd/src/wlan_hdd_main.c +++ b/core/hdd/src/wlan_hdd_main.c @@ -4500,6 +4500,8 @@ bool hdd_check_for_opened_interfaces(hdd_context_t *hdd_ctx) status = hdd_get_front_adapter(hdd_ctx, &adapter_node); while ((NULL != adapter_node) && (QDF_STATUS_SUCCESS == status)) { if (test_bit(DEVICE_IFACE_OPENED, + &adapter_node->pAdapter->event_flags) || + test_bit(SME_SESSION_OPENED, &adapter_node->pAdapter->event_flags)) { hdd_debug("Still other ifaces are up cannot close modules"); close_modules = false; -- cgit v1.2.3