diff options
| author | Himanshu Agarwal <himanaga@qti.qualcomm.com> | 2016-07-12 18:50:02 +0530 |
|---|---|---|
| committer | Anjaneedevi Kapparapu <akappa@codeaurora.org> | 2016-07-13 14:53:13 +0530 |
| commit | aff34bc81822a80e144858e7292c3898ac7b0ff2 (patch) | |
| tree | 0c8779ca3d7022ebfb7e9f4c66eb3539c30c9762 | |
| parent | eb2d58e7e33daf64387af5faa68fa2a102408fd6 (diff) | |
qcacld-2.0: Move NBUF_UPDATE_TX_PKT_COUNT before freeing netbuf
Move NBUF_UPDATE_TX_PKT_COUNT in ol_tx_completion_handler to make
sure that netbuf is not accessed after it is freed.
Change-Id: Ifba9de788b11ce8cb323827d10f8005029609231
CRs-fixed: 1040612
| -rw-r--r-- | CORE/CLD_TXRX/TXRX/ol_tx_send.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/CORE/CLD_TXRX/TXRX/ol_tx_send.c b/CORE/CLD_TXRX/TXRX/ol_tx_send.c index 8616dea07206..f6b187eda600 100644 --- a/CORE/CLD_TXRX/TXRX/ol_tx_send.c +++ b/CORE/CLD_TXRX/TXRX/ol_tx_send.c @@ -555,6 +555,7 @@ ol_tx_completion_handler( tx_desc = td_array[tx_desc_id].tx_desc; tx_desc->status = status; netbuf = tx_desc->netbuf; + NBUF_UPDATE_TX_PKT_COUNT(netbuf, NBUF_TX_PKT_FREE); DPTRACE(adf_dp_trace_ptr(netbuf, ADF_DP_TRACE_FREE_PACKET_PTR_RECORD, adf_nbuf_data_addr(netbuf), @@ -580,7 +581,6 @@ ol_tx_completion_handler( pdev, tx_desc, tx_descs, netbuf, lcl_freelist, tx_desc_last, status); } - NBUF_UPDATE_TX_PKT_COUNT(netbuf, NBUF_TX_PKT_FREE); #ifdef QCA_SUPPORT_TXDESC_SANITY_CHECKS tx_desc->pkt_type = 0xff; #ifdef QCA_COMPUTE_TX_DELAY |