laforge submitted this change.

View Change

Approvals: pespin: Looks good to me, approved laforge: Looks good to me, but someone else must approve Jenkins Builder: Verified
TCAP: UDTS routing: stop overwriting ASP when routing UDTS

The UDTS routing decision is final and should not go to the tcap routing
a second time as a normal packet.

Change-Id: I1c4eaaf352d505970e55e04c605c100d64aaaa34
---
M src/tcap_as_loadshare.c
1 file changed, 2 insertions(+), 0 deletions(-)

diff --git a/src/tcap_as_loadshare.c b/src/tcap_as_loadshare.c
index 8cc32d6..b699281 100644
--- a/src/tcap_as_loadshare.c
+++ b/src/tcap_as_loadshare.c
@@ -614,6 +614,8 @@
asp = tcap_as_asp_find_by_tcap_id(as, &called_addr, &calling_addr, *otid);
if (!asp)
LOGPAS(as, DLTCAP, LOGL_NOTICE, "Could not find an ASP for UDTS with otid %u/dtid %u\n", parsed.otid, parsed.dtid);
+
+ goto out_free_sua;
}

/* TCAP messages towards the IPA nodes */

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

Gerrit-MessageType: merged
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: master
Gerrit-Change-Id: I1c4eaaf352d505970e55e04c605c100d64aaaa34
Gerrit-Change-Number: 43193
Gerrit-PatchSet: 7
Gerrit-Owner: lynxis lazus <lynxis@fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>