diff options
| author | Amarnath Hullur Subramanyam <amarnath@codeaurora.org> | 2015-06-24 19:56:55 +0530 |
|---|---|---|
| committer | Vineeta Srivastava <vsrivastava@google.com> | 2015-07-09 11:59:08 -0700 |
| commit | 44273e6b215d69148af9f2cfab11c5eae9dcd233 (patch) | |
| tree | 5e2ca7b794697937ffe4d1c156de58ff088f099f | |
| parent | 9f6518676d85518024689fa0144c1c32c75cb3fe (diff) | |
| download | wlan-44273e6b215d69148af9f2cfab11c5eae9dcd233.tar.gz | |
Wifi-HAL: Fix possible page faults in parsing debug events
Fix wrong pointers passed to addLoggerTLV() to avoid possible
page faults.
Change-Id: I70b67f950d9036199ff13901d44285cfc250e1a2
| -rw-r--r-- | qcwcn/wifi_hal/wifilogger_diag.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/qcwcn/wifi_hal/wifilogger_diag.cpp b/qcwcn/wifi_hal/wifilogger_diag.cpp index 9c52a0f..a666ff2 100644 --- a/qcwcn/wifi_hal/wifilogger_diag.cpp +++ b/qcwcn/wifi_hal/wifilogger_diag.cpp @@ -378,8 +378,8 @@ static wifi_error process_extscan_event(hal_info *info, u32 id, pConnectEvent->event = WIFI_EVENT_G_SCAN_STOP; pExtScanStop = (wlan_ext_scan_feature_stop_payload_type *)buf; pTlv = addLoggerTlv(WIFI_TAG_REQUEST_ID, - sizeof(wlan_ext_scan_feature_stop_payload_type), - (u8 *)&pExtScanStop, pTlv); + sizeof(pExtScanStop->request_id), + (u8 *)&pExtScanStop->request_id, pTlv); tot_len += sizeof(tlv_log) + sizeof(wlan_ext_scan_feature_stop_payload_type); } @@ -502,7 +502,7 @@ static wifi_error process_addba_failed_event(hal_info *info, pTlv = addLoggerTlv(WIFI_TAG_VENDOR_SPECIFIC, sizeof(addba_failed_vendor_data_t), - (u8 *)&pAddBAFailed, pTlv); + (u8 *)&addBAFailedVenData, pTlv); tot_len += sizeof(tlv_log) + sizeof(addba_failed_vendor_data_t); status = update_connectivity_ring_buf(info, pRingBufferEntry, tot_len); |