[MERGED] openbsc[master]: Use define for limit on number of BTS features

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Max gerrit-no-reply at lists.osmocom.org
Wed May 10 07:49:44 UTC 2017


Max has submitted this change and it was merged.

Change subject: Use define for limit on number of BTS features
......................................................................


Use define for limit on number of BTS features

Change-Id: Ief81db0b3653fec180e60776ff9bcb0ac1357217
Related: OS#1614
---
M openbsc/include/openbsc/gsm_data_shared.h
1 file changed, 3 insertions(+), 1 deletion(-)

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



diff --git a/openbsc/include/openbsc/gsm_data_shared.h b/openbsc/include/openbsc/gsm_data_shared.h
index 0586d8a..5c96a62 100644
--- a/openbsc/include/openbsc/gsm_data_shared.h
+++ b/openbsc/include/openbsc/gsm_data_shared.h
@@ -66,6 +66,8 @@
 
 #define MAX_VERSION_LENGTH 64
 
+#define MAX_BTS_FEATURES 128
+
 enum gsm_hooks {
 	GSM_HOOK_NM_SWLOAD,
 	GSM_HOOK_RR_PAGING,
@@ -533,7 +535,7 @@
 	struct tlv_definition nm_att_tlvdef;
 
 	struct bitvec features;
-	uint8_t _features_data[128/8];
+	uint8_t _features_data[MAX_BTS_FEATURES/8];
 };
 
 enum gsm_bts_features {

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ief81db0b3653fec180e60776ff9bcb0ac1357217
Gerrit-PatchSet: 2
Gerrit-Project: openbsc
Gerrit-Branch: master
Gerrit-Owner: Max <msuraev at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Max <msuraev at sysmocom.de>



More information about the gerrit-log mailing list