pespin submitted this change.

View Change

Approvals: osmith: Looks good to me, approved Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve
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(-)

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 change 42053. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: merged
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: master
Gerrit-Change-Id: Iee339b49d4b73a9afde60c4b8f6f5a03ad8a822d
Gerrit-Change-Number: 42053
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>