From 484fc3875b80769de84469f3965976d699e00180 Mon Sep 17 00:00:00 2001 From: Harald Welte Date: Tue, 16 Nov 2021 11:18:52 +0100 Subject: osmo_ss7: properly unlink rate_counter on AS/ASP free When introducing rate_couters, I forgot to call rate_ctr_group_free(). I thought free'ing the parent object via talloc is sufficient, but that obviously misses the point that rate_counters have an internal linked list from which they must be unlinked. Change-Id: I8d27f025c22776d0153d867e36c073ef716eb974 --- src/osmo_ss7.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/osmo_ss7.c b/src/osmo_ss7.c index 1cb6c82..b4c0b51 100644 --- a/src/osmo_ss7.c +++ b/src/osmo_ss7.c @@ -1111,6 +1111,7 @@ void osmo_ss7_as_destroy(struct osmo_ss7_as *as) as->inst = NULL; llist_del(&as->list); + rate_ctr_group_free(as->ctrg); talloc_free(as); } @@ -1574,6 +1575,7 @@ void osmo_ss7_asp_destroy(struct osmo_ss7_asp *asp) /* unlink from ss7_instance */ asp->inst = NULL; llist_del(&asp->list); + rate_ctr_group_free(asp->ctrg); /* release memory */ talloc_free(asp); } -- cgit v1.2.3