ionic: Don't check null when calling vfree()

vfree() checks for null internally, so there's no need to
check in the caller. So, always vfree() on variables
allocated with valloc(). If the variables are never
alloc'd vfree() is still safe.

Signed-off-by: Brett Creeley <brett.creeley@amd.com>
Signed-off-by: Shannon Nelson <shannon.nelson@amd.com>
Reviewed-by: Florian Fainelli <florian.fainelli@broadcom.com>
Reviewed-by: Rahul Rameshbabu <rrameshbabu@nvidia.com>
Link: https://lore.kernel.org/r/20231204210936.16587-4-shannon.nelson@amd.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>

diff --git a/drivers/net/ethernet/pensando/ionic/ionic_lif.c b/drivers/net/ethernet/pensando/ionic/ionic_lif.c
index a5e6b1e..6842a31 100644 (file)
--- a/drivers/net/ethernet/pensando/ionic/ionic_lif.c
+++ b/drivers/net/ethernet/pensando/ionic/ionic_lif.c
@@ -424,14 +424,10 @@ static void ionic_qcq_free(struct ionic_lif *lif, struct ionic_qcq *qcq)
 
        ionic_qcq_intr_free(lif, qcq);
 
-       if (qcq->cq.info) {
-               vfree(qcq->cq.info);
-               qcq->cq.info = NULL;
-       }
-       if (qcq->q.info) {
-               vfree(qcq->q.info);
-               qcq->q.info = NULL;
-       }
+       vfree(qcq->cq.info);
+       qcq->cq.info = NULL;
+       vfree(qcq->q.info);
+       qcq->q.info = NULL;
 }
 
 void ionic_qcqs_free(struct ionic_lif *lif)