qcacld-3.0: Remove op_string enum to avoid out of bounds access
Fix potential buffer overflow by array index out of bounds.
We need to add a new op_string entry whenever a new opcode added.
Otherwise, the op_string entry for the additional opcodes could
be accessed out-of-bounds.
Change-Id: Ic09b19916b63d8bb83bf278361b8519268ee1787
CRs-Fixed: 2005340
diff --git a/core/hdd/src/wlan_hdd_ipa.c b/core/hdd/src/wlan_hdd_ipa.c
index 76cb0a2..25d9095 100644
--- a/core/hdd/src/wlan_hdd_ipa.c
+++ b/core/hdd/src/wlan_hdd_ipa.c
@@ -593,14 +593,6 @@
}
};
-static const char *op_string[] = {
- "TX_SUSPEND",
- "TX_RESUME",
- "RX_SUSPEND",
- "RX_RESUME",
- "STATS",
-};
-
static struct hdd_ipa_priv *ghdd_ipa;
/* Local Function Prototypes */
@@ -1898,7 +1890,7 @@
hdd_ipa = (struct hdd_ipa_priv *)hdd_ctx->hdd_ipa;
HDD_IPA_DP_LOG(QDF_TRACE_LEVEL_DEBUG,
- "%s, OPCODE %s", __func__, op_string[msg->op_code]);
+ "OPCODE=%d", msg->op_code);
if ((HDD_IPA_UC_OPCODE_TX_RESUME == msg->op_code) ||
(HDD_IPA_UC_OPCODE_RX_RESUME == msg->op_code)) {