From 014fe30ac93c84d810e0209bc7f0d3b7a1bbaee3 Mon Sep 17 00:00:00 2001
From: Sakthivel Velumani <velumani@eurecom.fr>
Date: Tue, 30 Aug 2022 15:49:27 -0400
Subject: [PATCH] Remove xer prints from e1ap and gcc warning fixes

---
 openair2/E1AP/e1ap_common.c   | 6 +++---
 openair2/E1AP/e1ap_common.h   | 2 ++
 openair2/GNB_APP/gnb_config.c | 8 ++++----
 3 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/openair2/E1AP/e1ap_common.c b/openair2/E1AP/e1ap_common.c
index fc77a22c53a..4ca53c9a205 100644
--- a/openair2/E1AP/e1ap_common.c
+++ b/openair2/E1AP/e1ap_common.c
@@ -164,7 +164,7 @@ int e1ap_decode_unsuccessful_outcome(E1AP_E1AP_PDU_t *pdu) {
   return 0;
 }
 
-int asn1_xer_print_e1ap = 1;
+int asn1_xer_print;
 
 int e1ap_decode_pdu(E1AP_E1AP_PDU_t *pdu, const uint8_t *const buffer, uint32_t length) {
   asn_dec_rval_t dec_ret;
@@ -177,7 +177,7 @@ int e1ap_decode_pdu(E1AP_E1AP_PDU_t *pdu, const uint8_t *const buffer, uint32_t
                         0,
                         0);
 
-  if (asn1_xer_print_e1ap) {
+  if (asn1_xer_print) {
     LOG_E(E1AP, "----------------- ASN1 DECODER PRINT START----------------- \n");
     xer_fprint(stdout, &asn_DEF_E1AP_E1AP_PDU, pdu);
     LOG_E(E1AP, "----------------- ASN1 DECODER PRINT END ----------------- \n");
@@ -209,7 +209,7 @@ int e1ap_decode_pdu(E1AP_E1AP_PDU_t *pdu, const uint8_t *const buffer, uint32_t
 int e1ap_encode_send(E1_t type, instance_t instance, E1AP_E1AP_PDU_t *pdu, uint16_t stream, const char *func) {
   DevAssert(pdu != NULL);
 
-  if (asn1_xer_print_e1ap) {
+  if (asn1_xer_print) {
     LOG_E(E1AP, "----------------- ASN1 ENCODER PRINT START ----------------- \n");
     xer_fprint(stdout, &asn_DEF_E1AP_E1AP_PDU, pdu);
     LOG_E(E1AP, "----------------- ASN1 ENCODER PRINT END----------------- \n");
diff --git a/openair2/E1AP/e1ap_common.h b/openair2/E1AP/e1ap_common.h
index 46d6524a7a4..6c57def40f3 100644
--- a/openair2/E1AP/e1ap_common.h
+++ b/openair2/E1AP/e1ap_common.h
@@ -62,6 +62,8 @@
 #include <E1AP_Dynamic5QIDescriptor.h>
 #include <E1AP_T-ReorderingTimer.h>
 
+extern int asn1_xer_print;
+
 int e1ap_decode_pdu(E1AP_E1AP_PDU_t *pdu, const uint8_t *const buffer, uint32_t length);
 
 e1ap_upcp_inst_t *getCxtE1(E1_t type, instance_t instance);
diff --git a/openair2/GNB_APP/gnb_config.c b/openair2/GNB_APP/gnb_config.c
index 9b4fcb289da..f08ab4474f4 100644
--- a/openair2/GNB_APP/gnb_config.c
+++ b/openair2/GNB_APP/gnb_config.c
@@ -1288,16 +1288,16 @@ static int get_NGU_S1U_addr(char *addr, int *port) {
 }
 
 instance_t RCconfig_nr_gtpu(void) {
-  char address[160];
+  char address[64];
   int port;
   int ret = get_NGU_S1U_addr(address, &port);
-  instance_t ret_inst;
+  instance_t ret_inst = 0;
   if (!ret) {
     eth_params_t IPaddr;
     IPaddr.my_addr = address;
     IPaddr.my_portd = port;
     openAddr_t tmp= {0};
-    strncpy(tmp.originHost, IPaddr.my_addr, sizeof(tmp.originHost)-1);
+    strcpy(tmp.originHost, IPaddr.my_addr);
     sprintf(tmp.originService, "%d",  IPaddr.my_portd);
     strcpy(tmp.destinationService, tmp.originService);
     LOG_I(GTPU,"Configuring GTPu address : %s, port : %s\n", tmp.originHost, tmp.originService);
@@ -1848,7 +1848,7 @@ int RCconfig_NR_CU_E1(MessageDef *msg_p, uint32_t i) {
     strcpy(e1Setup->CUUP_e1_ip_address.ipv4_address, *(GNBE1ParamList.paramarray[0][GNB_CONFIG_E1_IPV4_ADDRESS_CUUP].strptr));
     e1Setup->CUUP_e1_ip_address.ipv4 = 1;
     e1Setup->port_cuup = *GNBE1ParamList.paramarray[0][GNB_CONFIG_E1_PORT_CUUP].uptr;
-    char N3Addr[160];
+    char N3Addr[64];
     int N3Port;
     if (!get_NGU_S1U_addr(N3Addr, &N3Port)) {;
       inet_pton(AF_INET,
-- 
GitLab