From 16589039819eaee4f3ebcb27c867b916ebc160d8 Mon Sep 17 00:00:00 2001 From: Jia Ding Date: Fri, 10 Feb 2017 17:08:36 +0800 Subject: qcacld-2.0: Fix possible null pointer dereference In hdd_tx_timeout(), adf_dp_trace() is called with nbuf set to NULL. In this case, ADF_NBUF_CB_DP_TRACE_PRINT() will cause a NULL pointer dereference. Fix is to add a NULL check for nbuf. Change-Id: I6445e572735a85c0de6321f65216b36dab89b2e8 CRs-Fixed: 2004984 --- CORE/SERVICES/COMMON/adf/adf_trace.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CORE/SERVICES/COMMON/adf/adf_trace.c b/CORE/SERVICES/COMMON/adf/adf_trace.c index 5696d997b406..4a7b30aea52c 100644 --- a/CORE/SERVICES/COMMON/adf/adf_trace.c +++ b/CORE/SERVICES/COMMON/adf/adf_trace.c @@ -813,7 +813,7 @@ void adf_dp_trace(adf_nbuf_t nbuf, enum ADF_DP_TRACE_ID code, return; adf_dp_add_record(code, data, size, - ADF_NBUF_CB_DP_TRACE_PRINT(nbuf)); + nbuf ? ADF_NBUF_CB_DP_TRACE_PRINT(nbuf) : false); } /** -- cgit v1.2.3