Vadim Yanitskiy has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/13897 )

Change subject: hlr.c: fix: also store the session state in read_cb_forward()
......................................................................

hlr.c: fix: also store the session state in read_cb_forward()

If the session state is not set (OSMO_GSUP_SESSION_STATE_NONE),
osmo_gsup_encode() would omit the OSMO_GSUP_SESSION_ID_IE.

Fix for Change-Id: Ia4f345abc877baaf0a8f73b8988e6514d9589bf5
Change-Id: Idcd209a59d1ee5230104f3101740140d366b0646
---
M src/hlr.c
1 file changed, 1 insertion(+), 0 deletions(-)

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



diff --git a/src/hlr.c b/src/hlr.c
index 8078db0..38422c8 100644
--- a/src/hlr.c
+++ b/src/hlr.c
@@ -466,6 +466,7 @@
        gsup_err->destination_name = talloc_memdup(gsup_err, 
gsup->destination_name, gsup->destination_name_len);
        gsup_err->destination_name_len = gsup->destination_name_len;
        gsup_err->message_type = OSMO_GSUP_MSGT_E_ROUTING_ERROR;
+       gsup_err->session_state = gsup->session_state;
        gsup_err->session_id = gsup->session_id;
        gsup_err->source_name = talloc_memdup(gsup_err, gsup->source_name, 
gsup->source_name_len);
        gsup_err->source_name_len = gsup->source_name_len;

--
To view, visit https://gerrit.osmocom.org/13897
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-hlr
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Idcd209a59d1ee5230104f3101740140d366b0646
Gerrit-Change-Number: 13897
Gerrit-PatchSet: 1
Gerrit-Owner: Vadim Yanitskiy <axilira...@gmail.com>
Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Vadim Yanitskiy <axilira...@gmail.com>

Reply via email to