pespin submitted this change.

View Change

Approvals: fixeria: Looks good to me, approved Jenkins Builder: Verified osmith: Looks good to me, but someone else must approve daniel: Looks good to me, but someone else must approve
sccp_user: Fix dynamic ASP not associated to AS

Recent commit introduced a regression when applying
default_layer_manager to all ASPs in m3ua-mode ASP.
The call to osmo_ss7_as_add_asp() was moved to an inner code path but
forgot to add it in another code path.

Fixes: ce63d3b2027cefb8412179497100d14ed3fee0dd
Change-Id: I16371ef813c2e12993caae16a1626d10ae8ded54
---
M src/sccp_user.c
1 file changed, 1 insertion(+), 0 deletions(-)

diff --git a/src/sccp_user.c b/src/sccp_user.c
index b5b6b4b..cada057 100644
--- a/src/sccp_user.c
+++ b/src/sccp_user.c
@@ -652,6 +652,7 @@
ss7_asp_peer_set_hosts(&asp->cfg.local, asp, &default_local_ip, 1);
if (default_remote_ip)
ss7_asp_peer_set_hosts(&asp->cfg.remote, asp, &default_remote_ip, 1);
+ osmo_ss7_as_add_asp(as, asp->cfg.name);
/* Make sure proper defaults are applied if app didn't
provide specific default values, then restart the ASP: */
ss7_asp_restart_after_reconfigure(asp);

To view, visit change 39671. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: merged
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: master
Gerrit-Change-Id: I16371ef813c2e12993caae16a1626d10ae8ded54
Gerrit-Change-Number: 39671
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann@sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>