From 9a67c5e5b50451f19acf3cc493d3f53f0b0e145d Mon Sep 17 00:00:00 2001
From: Guido Casati <hello@guidocasati.com>
Date: Mon, 23 Sep 2024 10:40:52 +0200
Subject: [PATCH] Fix scoping issues for variable declarations in switch-case
 blocks (capture_pdu_session_establishment_accept_msg)

---
 .../ESM/MSG/PduSessionEstablishmentAccept.c    | 18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 deletions(-)

diff --git a/openair3/NAS/COMMON/ESM/MSG/PduSessionEstablishmentAccept.c b/openair3/NAS/COMMON/ESM/MSG/PduSessionEstablishmentAccept.c
index aee6d0e63d..a3c6039752 100644
--- a/openair3/NAS/COMMON/ESM/MSG/PduSessionEstablishmentAccept.c
+++ b/openair3/NAS/COMMON/ESM/MSG/PduSessionEstablishmentAccept.c
@@ -174,42 +174,47 @@ void capture_pdu_session_establishment_accept_msg(uint8_t *buffer, uint32_t msg_
         curPtr++; /* TS 24.008 10.5.7.3 */
         break;
 
-      case IEI_SNSSAI: /* Ommited */
+      case IEI_SNSSAI: {
         LOG_T(NAS, "PDU SESSION ESTABLISHMENT ACCEPT - Received S-NSSAI IE\n");
         uint8_t snssai_length = *curPtr;
         curPtr += (snssai_length + sizeof(snssai_length));
         break;
+      }
 
       case IEI_ALWAYSON_PDU: /* Ommited */
         LOG_T(NAS, "PDU SESSION ESTABLISHMENT ACCEPT - Received Always-on PDU Session indication IE\n");
         curPtr++;
         break;
 
-      case IEI_MAPPED_EPS: /* Ommited */
+      case IEI_MAPPED_EPS: {
         LOG_T(NAS, "PDU SESSION ESTABLISHMENT ACCEPT - Received Mapped EPS bearer context IE\n");
         uint16_t mapped_eps_length = getShort(curPtr);
         curPtr += mapped_eps_length;
         break;
+      }
 
-      case IEI_EAP_MSG: /* Ommited */
+      case IEI_EAP_MSG: {
         LOG_T(NAS, "PDU SESSION ESTABLISHMENT ACCEPT - Received EAP message IE\n");
         uint16_t eap_length = getShort(curPtr);
         curPtr += (eap_length + sizeof(eap_length));
         break;
+      }
 
-      case IEI_AUTH_QOS_DESC: /* Ommited */
+      case IEI_AUTH_QOS_DESC: {
         LOG_T(NAS, "PDU SESSION ESTABLISHMENT ACCEPT - Received Authorized QoS flow descriptions IE\n");
         psea_msg.qos_fd_ie.length = getShort(curPtr);
         curPtr += (psea_msg.qos_fd_ie.length + sizeof(psea_msg.qos_fd_ie.length));
         break;
+      }
 
-      case IEI_EXT_CONF_OPT: /* Ommited */
+      case IEI_EXT_CONF_OPT: {
         LOG_T(NAS, "PDU SESSION ESTABLISHMENT ACCEPT - Received Extended protocol configuration options IE\n");
         psea_msg.ext_pp_ie.length = getShort(curPtr);
         curPtr += (psea_msg.ext_pp_ie.length + sizeof(psea_msg.ext_pp_ie.length));
         break;
+      }
 
-      case IEI_DNN:
+      case IEI_DNN: {
         LOG_T(NAS, "PDU SESSION ESTABLISHMENT ACCEPT - Received DNN IE\n");
         psea_msg.dnn_ie.dnn_length = *curPtr++;
         char apn[APN_MAX_LEN];
@@ -223,6 +228,7 @@ void capture_pdu_session_establishment_accept_msg(uint8_t *buffer, uint32_t msg_
 
         curPtr = buffer + msg_length; // we force stop processing
         break;
+      }
 
       default:
         LOG_T(NAS, "PDU SESSION ESTABLISHMENT ACCEPT - Not known IE\n");