From cfccef5145a7f5c007900aab9105445ade973985 Mon Sep 17 00:00:00 2001 From: Hanumanth Reddy Pothula Date: Fri, 2 Dec 2016 14:42:17 +0530 Subject: qcacld-2.0: dereference ini config object after NULL check If unload is triggered after re-init sequence failure, there is a possibility of NULL pointer dereference while checking ini parameter values as cfg_ini will be set to NULL on re-init failure Add NULL check for ini config object, before dereferencing it. Change-Id: Id60f6c4bd48ce08c45244a99790eb7ef974f4eba CRs-Fixed: 1096698 --- CORE/HDD/src/wlan_hdd_cfg.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/CORE/HDD/src/wlan_hdd_cfg.c b/CORE/HDD/src/wlan_hdd_cfg.c index 702d9436446e..ffcb181d9f7c 100644 --- a/CORE/HDD/src/wlan_hdd_cfg.c +++ b/CORE/HDD/src/wlan_hdd_cfg.c @@ -6582,9 +6582,10 @@ VOS_STATUS hdd_set_idle_ps_config(hdd_context_t *pHddCtx, v_U32_t val) hdd_config_t *pConfig = pHddCtx->cfg_ini; VOS_STATUS status = VOS_STATUS_SUCCESS; - hddLog(LOG1, "hdd_set_idle_ps_config: Enter Val %d", val); + hddLog(LOG1, "hdd_set_idle_ps_config: Enter Val %d pconfig %p ", + val, pConfig); - if(pConfig->fIsImpsEnabled) + if(pConfig && pConfig->fIsImpsEnabled) { status = sme_SetIdlePowersaveConfig(pHddCtx->pvosContext, val); if(VOS_STATUS_SUCCESS != status) -- cgit v1.2.3