osmith submitted this change.

View Change

Approvals: Jenkins Builder: Verified
gsup_client_mux: set destination_name in error reply function

If the GSUP request message to which we are replying is an MT SMS
delivery from an SMSC relayed via OsmoHLR, we must set destination_name
in our reply - otherwise our reply won't make it back to the SMSC.

Related: OS#6135
Change-Id: I892fe87a733a78ed9d5761a8ce238caa135dea1e
---
M src/libmsc/gsup_client_mux.c
1 file changed, 16 insertions(+), 0 deletions(-)

diff --git a/src/libmsc/gsup_client_mux.c b/src/libmsc/gsup_client_mux.c
index 1900377..9a0dc16 100644
--- a/src/libmsc/gsup_client_mux.c
+++ b/src/libmsc/gsup_client_mux.c
@@ -150,6 +150,8 @@
.cause = cause,
.message_type = OSMO_GSUP_TO_MSGT_ERROR(gsup_orig->message_type),
.message_class = gsup_orig->message_class,
+ .destination_name = gsup_orig->source_name,
+ .destination_name_len = gsup_orig->source_name_len,

/* RP-Message-Reference is mandatory for SM Service */
.sm_rp_mr = gsup_orig->sm_rp_mr,

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

Gerrit-Project: osmo-msc
Gerrit-Branch: osmith/1.11.1
Gerrit-Change-Id: I892fe87a733a78ed9d5761a8ce238caa135dea1e
Gerrit-Change-Number: 34561
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: falconia <falcon@freecalypso.org>
Gerrit-MessageType: merged