laforge submitted this change.

View Change


Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved pespin: Looks good to me, but someone else must approve
HNBAP: use GSM23003_IMSI_MAX_DIGITS instead of magic number

Change-Id: I2ea0a6a93194da2efb768cd4245767301411eb41
---
M src/osmo-hnbgw/hnbgw_hnbap.c
1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/src/osmo-hnbgw/hnbgw_hnbap.c b/src/osmo-hnbgw/hnbgw_hnbap.c
index 6d3c8e0..fc06462 100644
--- a/src/osmo-hnbgw/hnbgw_hnbap.c
+++ b/src/osmo-hnbgw/hnbgw_hnbap.c
@@ -22,6 +22,7 @@
#include <osmocom/core/utils.h>
#include <osmocom/core/socket.h>
#include <osmocom/gsm/gsm48.h>
+#include <osmocom/gsm/protocol/gsm_23_003.h>
#include <osmocom/netif/stream.h>

#include <unistd.h>
@@ -564,7 +565,7 @@
HNBAP_Cause_t cause;
struct ue_context *ue;
struct ue_context *ue_allocated = NULL;
- char imsi[16];
+ char imsi[GSM23003_IMSI_MAX_DIGITS+1];
int rc;

rc = hnbap_decode_ueregisterrequesties(&ies, in);

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

Gerrit-Project: osmo-hnbgw
Gerrit-Branch: master
Gerrit-Change-Id: I2ea0a6a93194da2efb768cd4245767301411eb41
Gerrit-Change-Number: 36495
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge@osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged