[MERGED] libosmo-sccp[master]: m3ua: Remove inbound routing context before routing

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Harald Welte gerrit-no-reply at lists.osmocom.org
Mon Apr 10 11:26:57 UTC 2017


Harald Welte has submitted this change and it was merged.

Change subject: m3ua: Remove inbound routing context before routing
......................................................................


m3ua: Remove inbound routing context before routing

After verifying the routing context of an incoming M3UA message, remove
the routing context before passing into MTP routing.  In the forwarding
case, we might want to set a new routing context on the outbound link,
and we don't want the routing context IE to show up twice.

Change-Id: I7a534cb1da275369c70766c059aaae8157ce6833
---
M src/m3ua.c
1 file changed, 4 insertions(+), 0 deletions(-)

Approvals:
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/m3ua.c b/src/m3ua.c
index 1868388..92470d7 100644
--- a/src/m3ua.c
+++ b/src/m3ua.c
@@ -515,6 +515,10 @@
 	OSMO_ASSERT(dh);
 	m3ua_dh_to_xfer_param(&xua->mtp, dh);
 
+	/* remove ROUTE_CTX as in the routing case we want to add a new
+	 * routing context on the outbound side */
+	xua_msg_free_tag(xua, M3UA_IEI_ROUTE_CTX);
+
 	return m3ua_hmdc_rx_from_l2(asp->inst, xua);
 out_err:
 	if (err)

-- 
To view, visit https://gerrit.osmocom.org/2266
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I7a534cb1da275369c70766c059aaae8157ce6833
Gerrit-PatchSet: 7
Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Gerrit-Owner: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder



More information about the gerrit-log mailing list