Re: [PATCH v3 13/30] baseband/acc100: reset pointer after rte_free

[Maxime Coquelin]

On 10/12/22 04:53, Hernan Vargas wrote:

Set local pointer to NULL after rte_free.
This needs to be set explicitly since logic may check for null pointers.

Fixes: 060e7672930 ("baseband/acc100: add queue configuration")
Cc: sta...@dpdk.org

Signed-off-by: Hernan Vargas <hernan.var...@intel.com>

No new line

Reviewed-by: Maxime Coquelin <maxime.coque...@redhat.com>
---
  drivers/baseband/acc/rte_acc100_pmd.c | 3 +++
  1 file changed, 3 insertions(+)

diff --git a/drivers/baseband/acc/rte_acc100_pmd.c 
b/drivers/baseband/acc/rte_acc100_pmd.c
index c8dffda8a4..3914aa7814 100644
--- a/drivers/baseband/acc/rte_acc100_pmd.c
+++ b/drivers/baseband/acc/rte_acc100_pmd.c
@@ -610,6 +610,9 @@ acc100_dev_close(struct rte_bbdev *dev)
                rte_free(d->info_ring);
                rte_free(d->sw_rings_base);
                d->sw_rings_base = NULL;
+               d->tail_ptrs = NULL;
+               d->info_ring = NULL;
+               d->harq_layout = NULL;
        }
        /* Ensure all in flight HW transactions are completed */
        usleep(ACC_LONG_WAIT);