From 1dfef0346b21bbafc79be193eee543dea296e7f3 Mon Sep 17 00:00:00 2001 From: Soumya Bhat Date: Mon, 25 Jun 2018 12:34:32 +0530 Subject: [PATCH] qcacmn: Reset variables appropriately In existing code, variable used to track mgmt. and control TLV is getting reset only in success case. Add change to reset it appropriately. CRs-Fixed: 2248313 Change-Id: Ieac5dc215f441b0530013ea1e36abcba19294754 --- dp/wifi3.0/dp_htt.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/dp/wifi3.0/dp_htt.c b/dp/wifi3.0/dp_htt.c index 8422cb24d90e..c854876abbbc 100644 --- a/dp/wifi3.0/dp_htt.c +++ b/dp/wifi3.0/dp_htt.c @@ -2209,10 +2209,6 @@ dp_process_ppdu_stats_tx_mgmtctrl_payload_tlv(struct dp_pdev *pdev, WDI_NO_VAL, pdev->pdev_id); } - pdev->mgmtctrl_frm_info.mgmt_buf = NULL; - pdev->mgmtctrl_frm_info.mgmt_buf_len = 0; - pdev->mgmtctrl_frm_info.ppdu_id = 0; - return QDF_STATUS_E_ALREADY; } @@ -2594,6 +2590,10 @@ static bool dp_txrx_ppdu_stats_handler(struct dp_soc *soc, (pdev, htt_t2h_msg, pdev->mgmtctrl_frm_info.ppdu_id) != QDF_STATUS_SUCCESS) free_buf = false; + + pdev->mgmtctrl_frm_info.mgmt_buf = NULL; + pdev->mgmtctrl_frm_info.mgmt_buf_len = 0; + pdev->mgmtctrl_frm_info.ppdu_id = 0; } if (ppdu_info)