diff options
| author | Arif Hussain <arifhussain@codeaurora.org> | 2016-10-25 14:46:53 -0700 |
|---|---|---|
| committer | Gerrit - the friendly Code Review server <code-review@localhost> | 2016-10-29 17:22:31 -0700 |
| commit | cf6b960cb0fa6604f18a57f8a7e29f6aeb8d9542 (patch) | |
| tree | 11364f4fa1590af59f6b6de4673c9e5fcac10edd | |
| parent | 31576d985adceb6581bb7ba30a815739b73928c4 (diff) | |
qcacld-3.0: Fix null pointer dereference in wma_roam_scan_offload_mode
In wma_roam_scan_offload_mode function validate mem_alloc return pointer
to avoid NULL pointer dereference.
Change-Id: Ib2733346614ddb99ac329aa929a774db2c4325d6
CRs-Fixed: 1082162
(cherry picked from commit 97961da746e461c6f4f8ab3d77b269db77e5d2fa)
| -rw-r--r-- | core/wma/src/wma_scan_roam.c | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/core/wma/src/wma_scan_roam.c b/core/wma/src/wma_scan_roam.c index 47237e812a2d..8d28d4806bca 100644 --- a/core/wma/src/wma_scan_roam.c +++ b/core/wma/src/wma_scan_roam.c @@ -755,14 +755,17 @@ QDF_STATUS wma_roam_scan_offload_mode(tp_wma_handle wma_handle, struct roam_offload_scan_params *params = qdf_mem_malloc(sizeof(*params)); + if (!params) { + WMA_LOGE("%s: Failed to allocate scan params", __func__); + return QDF_STATUS_E_NOMEM; + } #ifdef WLAN_FEATURE_ROAM_OFFLOAD - int auth_mode = WMI_AUTH_NONE; + params->auth_mode = WMI_AUTH_NONE; if (roam_req) - auth_mode = e_csr_auth_type_to_rsn_authmode + params->auth_mode = e_csr_auth_type_to_rsn_authmode (roam_req->ConnectedNetwork.authentication, roam_req->ConnectedNetwork.encryption); - WMA_LOGD("%s : auth mode = %d", __func__, auth_mode); - params->auth_mode = auth_mode; + WMA_LOGD("%s : auth mode = %d", __func__, params->auth_mode); #endif /* WLAN_FEATURE_ROAM_OFFLOAD */ params->is_roam_req_valid = 0; |