From 5afc71f382049f30dddf61216db05f250ab47c00 Mon Sep 17 00:00:00 2001
From: Cedric Roux <cedric.roux@eurecom.fr>
Date: Fri, 20 Oct 2017 14:49:45 +0200
Subject: [PATCH] bugfix: fix featureGroup
- allocate memory, memory on stack does not survive the function
- avoid uint32_t, use a char buffer, for endianness neutrality
---
openair2/RRC/LITE/MESSAGES/asn1_msg.c | 19 +++++++++----------
1 file changed, 9 insertions(+), 10 deletions(-)
diff --git a/openair2/RRC/LITE/MESSAGES/asn1_msg.c b/openair2/RRC/LITE/MESSAGES/asn1_msg.c
index c378e476637..74b2dabd5ac 100644
--- a/openair2/RRC/LITE/MESSAGES/asn1_msg.c
+++ b/openair2/RRC/LITE/MESSAGES/asn1_msg.c
@@ -2577,18 +2577,17 @@ OAI_UECapability_t *fill_ue_capability(char *UE_EUTRA_Capability_xer_fname)
// UE_EUTRA_Capability->measParameters.bandListEUTRA.list.count = 0; // no measurements on other bands
// UE_EUTRA_Capability->featureGroupIndicators // null
- // featureGroup is mandatory for CMW tests
- // featureGroup is filled only for usim-test mode
- BIT_STRING_t *bit_string;
- uint32_t featrG;
- bit_string = CALLOC(1, sizeof(*bit_string));
- featrG = 0x04000800;
if(usim_test == 1)
{
- bit_string->buf = &featrG;
- bit_string->size = 4;
- bit_string->bits_unused = 0;
- UE_EUTRA_Capability->featureGroupIndicators = bit_string;
+ // featureGroup is mandatory for CMW tests
+ // featureGroup is filled only for usim-test mode
+ BIT_STRING_t *bit_string = CALLOC(1, sizeof(*bit_string));
+ char featrG[4] = { 0x00, 0x08, 0x00, 0x04 };
+ bit_string->buf = CALLOC(1, 4);
+ memcpy(bit_string->buf, featrG, 4);
+ bit_string->size = 4;
+ bit_string->bits_unused = 0;
+ UE_EUTRA_Capability->featureGroupIndicators = bit_string;
}
// UE_EUTRA_Capability->interRAT_Parameters // null
--
GitLab