pespin has submitted this change. (
https://gerrit.osmocom.org/c/libosmo-sigtran/+/42053?usp=email )
Change subject: ss7_asp: remove lm during ASP destroy
......................................................................
ss7_asp: remove lm during ASP destroy
Otherwise the xua_default_lm_fsm instance is freed with talloc_free()
instead of osmo_fsm_inst_free().
Change-Id: Iee339b49d4b73a9afde60c4b8f6f5a03ad8a822d
---
M src/ss7_asp.c
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
osmith: Looks good to me, approved
Jenkins Builder: Verified
laforge: Looks good to me, but someone else must approve
diff --git a/src/ss7_asp.c b/src/ss7_asp.c
index 4ca1fb4..ded2c67 100644
--- a/src/ss7_asp.c
+++ b/src/ss7_asp.c
@@ -758,6 +758,7 @@
osmo_stream_cli_destroy(asp->client);
if (asp->fi)
osmo_fsm_inst_term(asp->fi, OSMO_FSM_TERM_REQUEST, NULL);
+ osmo_ss7_asp_remove_default_lm(asp);
/* Unlink from all ASs we are part of.
* Some RKM-dynamically allocated ASs may be freed as a result from
this: */
--
To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/42053?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: master
Gerrit-Change-Id: Iee339b49d4b73a9afde60c4b8f6f5a03ad8a822d
Gerrit-Change-Number: 42053
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
