diff options
author | Ajit Vaishya <ajitv@codeaurora.org> | 2021-11-24 16:51:52 +0000 |
---|---|---|
committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2021-11-24 16:51:52 +0000 |
commit | df6958b32009eff68cf45e0b994636fdd8e5fd7f (patch) | |
tree | 2884f194843399e3cb4a85eb8c43985b6a905443 | |
parent | 198b6c7e2930810dc5724efda4c8c133e4eb9184 (diff) | |
parent | ef79d4cb49b7eb7f21f50a744018b088e6a0f7cb (diff) | |
download | wlan-df6958b32009eff68cf45e0b994636fdd8e5fd7f.tar.gz |
Wifi-Hal: Validate TX rate code before parsing it am: ef79d4cb49
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/qcom/wlan/+/16191795
Change-Id: Icb2f018a919a49f6bdab574c74e5dea5dfaf9024
-rw-r--r-- | qcwcn/wifi_hal/pkt_stats.h | 1 | ||||
-rw-r--r-- | qcwcn/wifi_hal/wifilogger_diag.cpp | 3 |
2 files changed, 3 insertions, 1 deletions
diff --git a/qcwcn/wifi_hal/pkt_stats.h b/qcwcn/wifi_hal/pkt_stats.h index 7cc64b8..d4348cd 100644 --- a/qcwcn/wifi_hal/pkt_stats.h +++ b/qcwcn/wifi_hal/pkt_stats.h @@ -49,6 +49,7 @@ #define PKTLOG_TYPE_MAX 12 #define BW_OFFSET 8 #define INVALID_RSSI 255 +#define INVALID_RATE_CODE 0xff /* Based on pkt log V2, this type of event will triggered */ #define PKTLOG_TYPE_PKT_SW_EVENT 10 diff --git a/qcwcn/wifi_hal/wifilogger_diag.cpp b/qcwcn/wifi_hal/wifilogger_diag.cpp index 436a42b..bdc9c7f 100644 --- a/qcwcn/wifi_hal/wifilogger_diag.cpp +++ b/qcwcn/wifi_hal/wifilogger_diag.cpp @@ -2338,7 +2338,8 @@ static wifi_error parse_stats_sw_event(hal_info *info, node_pkt_t.qos_ctrl = *((u8*)(node_pkt_data + QOS_CTRL_OFFSET)); rb_pkt_stats->tid = node_pkt_t.qos_ctrl & 0xF; rb_pkt_stats->MCS = get_tx_mcs_v1(node_pkt_data); - rb_pkt_stats->last_transmit_rate = get_rate_v1(rb_pkt_stats->MCS); + if ((rb_pkt_stats->MCS & INVALID_RATE_CODE) != INVALID_RATE_CODE) + rb_pkt_stats->last_transmit_rate = get_rate_v1(rb_pkt_stats->MCS); node_pkt_t.bmap_failed = *((u64*)(node_pkt_data + BMAP_FAILED_OFFSET)); node_pkt_t.bmap_enqueued = *((u64*)(node_pkt_data + BMAP_ENQUEUED_OFFSET)); |