From 09e10eb5e2a1b0a4aa540efa15059fc2811198b0 Mon Sep 17 00:00:00 2001 From: lifeng Date: Fri, 17 May 2019 13:00:12 +0800 Subject: [PATCH] qcacmn: Remove one possible reachable assertion Remove one possible reachable assertion in case tainted FW easily trigger the assert and perform a denial of service. Change-Id: I913d619ab6268c0a843d93ed35c63c79b96ac488 CRs-Fixed: 2428205 --- wmi/src/wmi_unified_ocb_tlv.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/wmi/src/wmi_unified_ocb_tlv.c b/wmi/src/wmi_unified_ocb_tlv.c index 357f96e979f7..cd55899c163a 100644 --- a/wmi/src/wmi_unified_ocb_tlv.c +++ b/wmi/src/wmi_unified_ocb_tlv.c @@ -744,10 +744,9 @@ static QDF_STATUS extract_ocb_dcc_stats_tlv(wmi_unified_t wmi_handle, if (fix_param->num_channels > ((WMI_SVC_MSG_MAX_SIZE - sizeof(*fix_param)) / sizeof(wmi_dcc_ndl_stats_per_channel)) || fix_param->num_channels > param_tlvs->num_stats_per_channel_list) { - WMI_LOGE("%s: too many channels:%d actual:%d", __func__, + WMI_LOGW("%s: too many channels:%d actual:%d", __func__, fix_param->num_channels, param_tlvs->num_stats_per_channel_list); - QDF_ASSERT(0); *resp = NULL; return QDF_STATUS_E_INVAL; }