diff options
| author | Sandeep Puligilla <spuligil@qti.qualcomm.com> | 2014-07-22 21:04:39 +0530 |
|---|---|---|
| committer | Akash Patel <c_akashp@qca.qualcomm.com> | 2014-07-28 19:45:27 -0700 |
| commit | 8cd7539e5af376235b297a97fdba6aaa0aa6edec (patch) | |
| tree | 8dcb65ea86abda533b99831eda9e92961e8451e5 | |
| parent | c29a7c0ec6e14b9abddeb13150a5240c2b5e5637 (diff) | |
wlan: CSR session validity check
Added NULL condition check to verify the received CSR session.
Change-Id: I1c7f259235856fcf1c897e90c7d253c8577b1c4b
CRS-Fixed: 692452
| -rw-r--r-- | CORE/SME/src/sme_common/sme_Api.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/CORE/SME/src/sme_common/sme_Api.c b/CORE/SME/src/sme_common/sme_Api.c index 0de9024b2db9..465ac37d00ad 100644 --- a/CORE/SME/src/sme_common/sme_Api.c +++ b/CORE/SME/src/sme_common/sme_Api.c @@ -13440,6 +13440,14 @@ VOS_STATUS sme_UpdateDSCPtoUPMapping( tHalHandle hHal, status = sme_AcquireGlobalLock( &pMac->sme ); if ( HAL_STATUS_SUCCESS( status ) ) { + pCsrSession = CSR_GET_SESSION( pMac, sessionId ); + if (pCsrSession == NULL) + { + VOS_TRACE(VOS_MODULE_ID_SME, VOS_TRACE_LEVEL_ERROR, + "%s: Session lookup fails for CSR session", __func__); + sme_ReleaseGlobalLock( &pMac->sme); + return eHAL_STATUS_FAILURE; + } if (!CSR_IS_SESSION_VALID( pMac, sessionId )) { VOS_TRACE(VOS_MODULE_ID_SME, VOS_TRACE_LEVEL_ERROR, @@ -13447,7 +13455,6 @@ VOS_STATUS sme_UpdateDSCPtoUPMapping( tHalHandle hHal, sme_ReleaseGlobalLock( &pMac->sme); return eHAL_STATUS_FAILURE; } - pCsrSession = CSR_GET_SESSION( pMac, sessionId ); pSession = peFindSessionByBssid( pMac, pCsrSession->connectedProfile.bssid, &peSessionId ); |