diff options
| author | Amarnath Hullur Subramanyam <amarnath@codeaurora.org> | 2015-09-23 20:29:51 -0700 |
|---|---|---|
| committer | Vineeta Srivastava <vsrivastava@google.com> | 2015-09-24 17:17:52 -0700 |
| commit | 3a6288afe9317768eef26f1e69e5fdbd97cd396e (patch) | |
| tree | 2e6c3f171bc829a2272e53d5d9e5004678c21c89 | |
| parent | e99ddb535a7c932891428d84a899497fc50b0f7e (diff) | |
| download | wlan-android-6.0.1_r27.tar.gz | |
Wifi-HAL: Fix incorrect firmware log entryandroid-n-preview-1android-6.0.1_r9android-6.0.1_r81android-6.0.1_r80android-6.0.1_r8android-6.0.1_r79android-6.0.1_r78android-6.0.1_r77android-6.0.1_r74android-6.0.1_r73android-6.0.1_r72android-6.0.1_r70android-6.0.1_r7android-6.0.1_r69android-6.0.1_r66android-6.0.1_r65android-6.0.1_r61android-6.0.1_r60android-6.0.1_r59android-6.0.1_r58android-6.0.1_r57android-6.0.1_r56android-6.0.1_r52android-6.0.1_r51android-6.0.1_r50android-6.0.1_r49android-6.0.1_r48android-6.0.1_r47android-6.0.1_r46android-6.0.1_r43android-6.0.1_r42android-6.0.1_r41android-6.0.1_r40android-6.0.1_r30android-6.0.1_r3android-6.0.1_r28android-6.0.1_r27android-6.0.1_r26android-6.0.1_r20android-6.0.1_r17android-6.0.1_r13android-6.0.1_r12android-6.0.1_r11android-6.0.1_r10android-6.0.1_r1android-6.0.0_r41marshmallow-mr3-releasemarshmallow-mr2-releasemarshmallow-mr1-releasemarshmallow-mr1-devmarshmallow-dr-dev
The commit "Add the fw diag msg hdr" introduced an error
while calculating the offset to the next buffer. This
caused incorrect firmware log entries in the ring buffer.
Change-Id: Iccce94c33a41f62f4e7c3477f603a6c8d33a1efa
CRs-Fixed: 913224
Bug: 24350330
| -rw-r--r-- | qcwcn/wifi_hal/wifilogger_diag.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/qcwcn/wifi_hal/wifilogger_diag.cpp b/qcwcn/wifi_hal/wifilogger_diag.cpp index 17443cf..742bc98 100644 --- a/qcwcn/wifi_hal/wifilogger_diag.cpp +++ b/qcwcn/wifi_hal/wifilogger_diag.cpp @@ -904,10 +904,9 @@ static wifi_error process_fw_diag_msg(hal_info *info, u8* buf, u16 length) case WLAN_DIAG_TYPE_MSG: { /* Length field is only one byte for WLAN_DIAG_TYPE_MSG */ - payloadlen = diag_msg_hdr->u.msg_hdr.payload_len + - sizeof(fw_diag_msg_hdr_t); + payloadlen = diag_msg_hdr->u.msg_hdr.payload_len; process_firmware_prints(info, (u8 *)diag_msg_hdr, - payloadlen); + payloadlen + sizeof(fw_diag_msg_hdr_t)); } break; default: |