diff options
| author | Padma, Santhosh Kumar <skpadma@codeaurora.org> | 2017-10-11 16:15:45 +0530 |
|---|---|---|
| committer | snandini <snandini@codeaurora.org> | 2017-10-12 11:30:35 -0700 |
| commit | 0e2c5d740dafec1d46a369b103a8e71ca4eaf54c (patch) | |
| tree | 606e38511f88378ec3022ec0684b207960884f1f | |
| parent | e34dce80d623bafa3c7a662499eb855e38aae42e (diff) | |
qcacld-3.0: Remove check against WMI_SVC_MSG_MAX_SIZE
Structure roam_offload_synch_ind has element hlp_data which is of
size FILS_MAX_HLP_DATA_LEN is greater than WMI_SVC_MSG_MAX_SIZE.
Hence, remove check for size of roam_offload_synch_ind against
WMI_SVC_MSG_MAX_SIZE to avoid roam failure. Also, modify check
for validity of vdev id.
Change-Id: I1b9cef08e1d847f27b7057abf7189ef0f867b92f
CRs-Fixed: 2124786
| -rw-r--r-- | core/wma/src/wma_scan_roam.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/core/wma/src/wma_scan_roam.c b/core/wma/src/wma_scan_roam.c index d9ed6e2678bb..24135b748a14 100644 --- a/core/wma/src/wma_scan_roam.c +++ b/core/wma/src/wma_scan_roam.c @@ -2616,7 +2616,7 @@ int wma_roam_synch_event_handler(void *handle, uint8_t *event, goto cleanup_label; } - if (synch_event->vdev_id > wma->max_bssid) { + if (synch_event->vdev_id >= wma->max_bssid) { WMA_LOGE("%s: received invalid vdev_id %d", __func__, synch_event->vdev_id); goto cleanup_label; @@ -2665,9 +2665,6 @@ int wma_roam_synch_event_handler(void *handle, uint8_t *event, (sizeof(*synch_event) + sizeof(wmi_channel) + sizeof(wmi_key_material) + sizeof(uint32_t))) goto cleanup_label; - if (sizeof(roam_offload_synch_ind) > - (WMI_SVC_MSG_MAX_SIZE - roam_synch_data_len)) - goto cleanup_label; roam_synch_data_len += sizeof(roam_offload_synch_ind); cds_host_diag_log_work(&wma->roam_ho_wl, |