diff options
| author | Jia Ding <jiad@codeaurora.org> | 2017-02-10 17:08:36 +0800 |
|---|---|---|
| committer | Gerrit - the friendly Code Review server <code-review@localhost> | 2017-02-14 04:20:14 -0800 |
| commit | 16589039819eaee4f3ebcb27c867b916ebc160d8 (patch) | |
| tree | 0aac0bfc8980fa1527c6d03669b717a152f95e12 | |
| parent | ad39e3745b1e4f0991a3f2609384fe253dee074d (diff) | |
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
| -rw-r--r-- | CORE/SERVICES/COMMON/adf/adf_trace.c | 2 |
1 files changed, 1 insertions, 1 deletions
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); } /** |