net/softnic: fix memory leak of meter policy

After the meter policies are created, they are not freed on device close. This patch fixes it. Fixes: 5f0d54f ("ethdev: add pre-defined meter policy API") Cc: [email protected] Signed-off-by: Dapeng Yu <[email protected]> Acked-by: Jasvinder Singh <[email protected]>
spdk · Oct 19, 2021 · 287ca31 · 287ca31
1 parent b314a4a
commit 287ca31
Showing 1 changed file with 12 additions and 0 deletions.
diff --git a/drivers/net/softnic/rte_eth_softnic_meter.c b/drivers/net/softnic/rte_eth_softnic_meter.c
@@ -52,6 +52,18 @@ softnic_mtr_free(struct pmd_internals *p)
 		TAILQ_REMOVE(&p->mtr.meter_profiles, mp, node);
 		free(mp);
 	}
+
+	/* Remove meter policies */
+	for ( ; ; ) {
+		struct softnic_mtr_meter_policy *mp;
+
+		mp = TAILQ_FIRST(&p->mtr.meter_policies);
+		if (mp == NULL)
+			break;
+
+		TAILQ_REMOVE(&p->mtr.meter_policies, mp, node);
+		free(mp);
+	}
 }
 
 struct softnic_mtr_meter_profile *