lynxis lazus submitted this change.

View Change

Approvals: pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified daniel: Looks good to me, approved
vlr: only set HLR number if field is present

Change-Id: I26dc4c00b81723cd7ead095a515269afd0f73f8e
---
M src/libvlr/vlr.c
1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/libvlr/vlr.c b/src/libvlr/vlr.c
index 12efdea..95b081a 100644
--- a/src/libvlr/vlr.c
+++ b/src/libvlr/vlr.c
@@ -439,8 +439,10 @@
gsup_msg.message_type = OSMO_GSUP_MSGT_PURGE_MS_REQUEST;

/* provide HLR number in case we know it */
- gsup_msg.hlr_enc_len = vsub->hlr.len;
- gsup_msg.hlr_enc = vsub->hlr.buf;
+ if (vsub->hlr.len) {
+ gsup_msg.hlr_enc_len = vsub->hlr.len;
+ gsup_msg.hlr_enc = vsub->hlr.buf;
+ }

gsup_msg.cn_domain = vlr_is_cs(vsub->vlr) ? OSMO_GSUP_CN_DOMAIN_CS : OSMO_GSUP_CN_DOMAIN_PS;


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

Gerrit-MessageType: merged
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: I26dc4c00b81723cd7ead095a515269afd0f73f8e
Gerrit-Change-Number: 39232
Gerrit-PatchSet: 2
Gerrit-Owner: lynxis lazus <lynxis@fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann@sysmocom.de>
Gerrit-Reviewer: lynxis lazus <lynxis@fe80.eu>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>