commit 90f43537d825b5c38f88cb838f66c83c792d5708
author Vinay Krishna Eranna <c_veran@qti.qualcomm.com> Mon Oct 27 16:45:54 2014 +0530
committer AnjaneeDevi Kapparapu <c_akappa@qti.qualcomm.com> Wed Oct 29 15:36:08 2014 +0530
tree 1bc4df2a07cad48885ab5131320327dca7affbc4
parent d3ce76ddc326b8762893c61375686c4a7de7ab57

wlan: validate the driver status during interface down

Validate the driver status before handling interface down in
hdd_stop function. This avoids any access to invalid data
during SSR or load/unload process.

Change-Id: Iff9b91cc51a11b2b0e556a5ba3cd94ecb77fa5f7
CRs-Fixed: 744263

CORE/HDD/src/wlan_hdd_main.c

diff --git a/CORE/HDD/src/wlan_hdd_main.c b/CORE/HDD/src/wlan_hdd_main.c
index 5541c8a..23dd2f7 100644
--- a/CORE/HDD/src/wlan_hdd_main.c
+++ b/CORE/HDD/src/wlan_hdd_main.c
@@ -5215,6 +5215,7 @@
 
 int __hdd_stop (struct net_device *dev)
 {
+   int ret = 0;
    hdd_adapter_t *pAdapter = WLAN_HDD_GET_PRIV_PTR(dev);
    hdd_context_t *pHddCtx;
    hdd_adapter_list_node_t *pAdapterNode = NULL, *pNext = NULL;
@@ -5230,12 +5231,14 @@
    }
    MTRACE(vos_trace(VOS_MODULE_ID_HDD, TRACE_CODE_HDD_OPEN_REQUEST,
                     pAdapter->sessionId, pAdapter->device_mode));
-   pHddCtx = (hdd_context_t*)pAdapter->pHddCtx;
-   if (NULL == pHddCtx)
+
+   pHddCtx = WLAN_HDD_GET_CTX(pAdapter);
+   ret = wlan_hdd_validate_context(pHddCtx);
+   if (ret)
    {
-      VOS_TRACE( VOS_MODULE_ID_HDD, VOS_TRACE_LEVEL_FATAL,
-         "%s: HDD context is Null", __func__);
-      return -ENODEV;
+      VOS_TRACE(VOS_MODULE_ID_HDD, VOS_TRACE_LEVEL_ERROR,
+                "%s: HDD context is not valid!", __func__);
+      return ret;
    }
 
    /* Nothing to be done if the interface is not opened */