Merge pull request #507 from daltonate/patch-1

Properly use critical section

Author: mluis1

src/mac/LoRaMac.c

@@ -1360,13 +1360,15 @@ static void ProcessRadioRxTimeout( void )
 
 static void LoRaMacHandleIrqEvents( void )
 {
-    LoRaMacRadioEvents_t events = LoRaMacRadioEvents;
+    LoRaMacRadioEvents_t events;
+
+    CRITICAL_SECTION_BEGIN( );
+    events = LoRaMacRadioEvents;
+    LoRaMacRadioEvents.Value = 0;
+    CRITICAL_SECTION_END( );
+
     if( events.Value != 0 )
     {
-        CRITICAL_SECTION_BEGIN( );
-        LoRaMacRadioEvents.Value = 0;
-        CRITICAL_SECTION_END( );
-
         if( events.Events.TxDone == 1 )
         {
             ProcessRadioTxDone( );

src/mac/LoRaMacClassB.c

@@ -1666,13 +1666,15 @@ void LoRaMacClassBSetMulticastPeriodicity( MulticastCtx_t* multicastChannel )
 void LoRaMacClassBProcess( void )
 {
 #ifdef LORAMAC_CLASSB_ENABLED
-    LoRaMacClassBEvents_t events = LoRaMacClassBEvents;
+    LoRaMacClassBEvents_t events;
+
+    CRITICAL_SECTION_BEGIN( );
+    events = LoRaMacClassBEvents;
+    LoRaMacClassBEvents.Value = 0;
+    CRITICAL_SECTION_END( );
+
     if( events.Value != 0 )
     {
-        CRITICAL_SECTION_BEGIN( );
-        LoRaMacClassBEvents.Value = 0;
-        CRITICAL_SECTION_END( );
-
         if( events.Events.Beacon == 1 )
         {
             LoRaMacClassBProcessBeacon( );