samsung-sm8650/android_kernel_samsung_sm8650-modules
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

qcacmn: Fix ref leak when monitor peer is null

Browse Source 
Free the reference when peer is not null and monitor peer
is null.

Change-Id: I81875fbdf3918346b8c4f297f61fc241e7888fd4
CRs-Fixed: 3277552
This commit is contained in:
Nandha Kishore Easwaran
2022-08-26 20:36:43 +05:30
committed by Madan Koyyalamudi
parent 640ab03114
commit 68ef6e5dff
1 changed files with 14 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
dp/wifi3.0/monitor/dp_mon.c
View File

@@ -3831,9 +3831,15 @@ dp_process_ppdu_stats_sch_cmd_status_tlv(struct dp_pdev *pdev,
* This check is to make sure peer is not deleted * This check is to make sure peer is not deleted
* after processing the TLVs. * after processing the TLVs.
*/ */
if (!peer || !peer->monitor_peer) if (!peer)
continue; continue;
if (!peer->monitor_peer) {
dp_peer_unref_delete(peer,
DP_MOD_ID_TX_PPDU_STATS);
continue;
}
mon_peer = peer->monitor_peer; mon_peer = peer->monitor_peer;
delay_ppdu = &mon_peer->delayed_ba_ppdu_stats; delay_ppdu = &mon_peer->delayed_ba_ppdu_stats;
start_tsf = ppdu_desc->ppdu_start_timestamp; start_tsf = ppdu_desc->ppdu_start_timestamp;
@@ -3888,9 +3894,15 @@ dp_process_ppdu_stats_sch_cmd_status_tlv(struct dp_pdev *pdev,
* This check is to make sure peer is not deleted * This check is to make sure peer is not deleted
* after processing the TLVs. * after processing the TLVs.
*/ */
if (!peer || !peer->monitor_peer) if (!peer)
continue; continue;
if (!peer->monitor_peer) {
dp_peer_unref_delete(peer,
DP_MOD_ID_TX_PPDU_STATS);
continue;
}
mon_peer = peer->monitor_peer; mon_peer = peer->monitor_peer;
if (ppdu_desc->user[i].completion_status != if (ppdu_desc->user[i].completion_status !=
HTT_PPDU_STATS_USER_STATUS_OK) { HTT_PPDU_STATS_USER_STATUS_OK) {