aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHarald Welte <laforge@osmocom.org>2021-11-16 11:18:52 +0100
committerHarald Welte <laforge@osmocom.org>2021-11-16 11:20:11 +0100
commit484fc3875b80769de84469f3965976d699e00180 (patch)
treecc3130116d3d99112df6b5866802b82573d66b77
parente9727688cd9b5b87bc708170081a71fe91993d21 (diff)
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
-rw-r--r--src/osmo_ss7.c2
1 files changed, 2 insertions, 0 deletions
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);
}