diff options
| author | Zhu Jianmin <jianminz@codeaurora.org> | 2017-09-07 13:33:50 +0800 |
|---|---|---|
| committer | snandini <snandini@codeaurora.org> | 2017-09-21 10:24:30 -0700 |
| commit | 3a663320751105d20853e59bf22f8ceaf3cccd52 (patch) | |
| tree | 739a3fcb0441c547f13b72444e8a97502bc1ab55 | |
| parent | a42c2da2dd1eb21b58ca638a547e09b9ef87b933 (diff) | |
qcacld-3.0: Avoid issue that NULL constant is dereferenced
NULL constant is unsuitable for sapEvent parameter since following
may be called:
sapEventInit(sapEvent);
vosStatus = sapFsm(sapContext, sapEvent);
Change-Id: I2ffc1be27f96bcbd793b2302e6c5920b5d30073b
CRs-Fixed: 2111290
| -rw-r--r-- | core/sap/src/sap_module.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/core/sap/src/sap_module.c b/core/sap/src/sap_module.c index 5e0fe698b20d..79eb041a0d07 100644 --- a/core/sap/src/sap_module.c +++ b/core/sap/src/sap_module.c @@ -3588,6 +3588,7 @@ wlansap_acs_chselect(void *pvos_gctx, tHalHandle h_hal = NULL; QDF_STATUS qdf_status = QDF_STATUS_E_FAILURE; tpAniSirGlobal pmac = NULL; + tWLAN_SAPEvent sapEvent; /* State machine event */ sap_context = CDS_GET_SAP_CB(pvos_gctx); if (NULL == sap_context) { @@ -3639,7 +3640,7 @@ wlansap_acs_chselect(void *pvos_gctx, * different scan callback fucntion to process * the results pre start BSS. */ - qdf_status = sap_goto_channel_sel(sap_context, NULL, true, false); + qdf_status = sap_goto_channel_sel(sap_context, &sapEvent, true, false); if (QDF_STATUS_E_ABORTED == qdf_status) { QDF_TRACE(QDF_MODULE_ID_SAP, QDF_TRACE_LEVEL_ERROR, |