Harald Welte has submitted this change and it was merged.

Change subject: hnbgw: hnb info: record MCC, MNC, show on 'show hnb'
......................................................................


hnbgw: hnb info: record MCC, MNC, show on 'show hnb'

Before, MCC and MNC were always reported as zero, now the output of 'show hnb 
all' looks like:

  OsmoHNBGW> show hnb all
  HNB "[email protected]" MCC 901 MNC 70 LAC 11111 RAC 99 SAC 
65535 CID 1048575
     HNBAP ID 0 RUA ID 0

Change-Id: Iae094b36fa1cf18e07ed33914b9425368d7cd34b
---
M src/Makefile.am
M src/hnbgw_hnbap.c
2 files changed, 2 insertions(+), 3 deletions(-)

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



diff --git a/src/Makefile.am b/src/Makefile.am
index ab772a4..85bfad6 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -67,7 +67,7 @@
                     hnbgw_vty.c \
                     context_map.c hnbgw_cn.c
 
-osmo_hnbgw_LDADD = $(OSMOCORE_LIBS) $(OSMOVTY_LIBS) \
+osmo_hnbgw_LDADD = $(OSMOCORE_LIBS) $(OSMOGSM_LIBS) $(OSMOVTY_LIBS) \
                   $(ASN1C_LIBS) $(OSMOSIGTRAN_LIBS) \
                   $(OSMONETIF_LIBS) \
                   hnbap/libosmo-asn1-hnbap.a rua/libosmo-asn1-rua.a \
diff --git a/src/hnbgw_hnbap.c b/src/hnbgw_hnbap.c
index 7c1e239..058a42a 100644
--- a/src/hnbgw_hnbap.c
+++ b/src/hnbgw_hnbap.c
@@ -380,8 +380,7 @@
        ctx->id.sac = asn1str_to_u16(&ies.sac);
        ctx->id.rac = asn1str_to_u8(&ies.rac);
        ctx->id.cid = asn1bitstr_to_u28(&ies.cellIdentity);
-       //ctx->id.mcc FIXME
-       //ctx->id.mnc FIXME
+       gsm48_mcc_mnc_from_bcd(ies.plmNidentity.buf, &ctx->id.mcc, 
&ctx->id.mnc);
 
        DEBUGP(DHNBAP, "HNB-REGISTER-REQ from %s\n", ctx->identity_info);
 

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iae094b36fa1cf18e07ed33914b9425368d7cd34b
Gerrit-PatchSet: 1
Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr <[email protected]>
Gerrit-Reviewer: Harald Welte <[email protected]>
Gerrit-Reviewer: Jenkins Builder

Reply via email to