Selaa lähdekoodia

qcacmn: Fix NULL pointer dereference

In dp_rx_mon_status_buffers_replenish after replenish of all
mon status ring rx buffers there is log in which function
local desc list is accessed. But after successful replenish of
all the available free desc, free desc list will be NULL. So
accessing the free desc list in the debug log leads to NULL
pointer dereference. In this change remove the debug log as
before that enough logging is in place to check how many buffers
are replenished.

Change-Id: I306aa52f6f25d9b43afb8548b8c03467aa9f27de
CRs-Fixed: 2366585
Sravan Kumar Kairam 6 vuotta sitten
vanhempi
sitoutus
099671a852
1 muutettua tiedostoa jossa 0 lisäystä ja 5 poistoa
  1. 0 5
      dp/wifi3.0/dp_rx_mon_status.c

+ 0 - 5
dp/wifi3.0/dp_rx_mon_status.c

@@ -953,11 +953,6 @@ QDF_STATUS dp_rx_mon_status_buffers_replenish(struct dp_soc *dp_soc,
 	QDF_TRACE(QDF_MODULE_ID_DP, QDF_TRACE_LEVEL_DEBUG,
 		"%d rx desc added back to free list", num_desc_to_free);
 
-	QDF_TRACE(QDF_MODULE_ID_DP, QDF_TRACE_LEVEL_DEBUG,
-		"[%s][%d] desc_list=%pK, tail=%pK rx_desc=%pK, cookie=%d",
-		__func__, __LINE__, desc_list, tail, &(*desc_list)->rx_desc,
-		(*desc_list)->rx_desc.cookie);
-
 	/*
 	 * add any available free desc back to the free list
 	 */