Exclude LmhpComplianceTxProcess function call from the critical section.

mluis1 · mluis1 · commit 4a2088c0e920 · 2019-07-09T11:02:47.000+02:00
diff --git a/src/apps/LoRaMac/common/LmHandler/packages/LmhpCompliance.c b/src/apps/LoRaMac/common/LmHandler/packages/LmhpCompliance.c
@@ -473,13 +473,16 @@ static void LmhpComplianceOnMcpsIndication( McpsIndication_t* mcpsIndication )
 
 static void LmhpComplianceProcess( void )
 {
+    bool isPending;
+
     CRITICAL_SECTION_BEGIN( );
-    if( ComplianceTestState.TxPending == true )
+    isPending = ComplianceTestState.TxPending;
+    ComplianceTestState.TxPending = false;
+    CRITICAL_SECTION_END( );
+    if( isPending == true )
     {
         LmhpComplianceTxProcess( );
-        ComplianceTestState.TxPending = false;
     }
-    CRITICAL_SECTION_END( );
 }
 
 static void OnComplianceTxNextPacketTimerEvent( void* context )

Original file line number	Diff line number	Diff line change
`@@ -473,13 +473,16 @@ static void LmhpComplianceOnMcpsIndication( McpsIndication_t* mcpsIndication )`
`473`	`473`
`474`	`474`	`static void LmhpComplianceProcess( void )`
`475`	`475`	`{`
	`476`	`+ bool isPending;`
	`477`	`+`
`476`	`478`	`CRITICAL_SECTION_BEGIN( );`
`477`		`- if( ComplianceTestState.TxPending == true )`
	`479`	`+ isPending = ComplianceTestState.TxPending;`
	`480`	`+ ComplianceTestState.TxPending = false;`
	`481`	`+ CRITICAL_SECTION_END( );`
	`482`	`+ if( isPending == true )`
`478`	`483`	`{`
`479`	`484`	`LmhpComplianceTxProcess( );`
`480`		`- ComplianceTestState.TxPending = false;`
`481`	`485`	`}`
`482`		`- CRITICAL_SECTION_END( );`
`483`	`486`	`}`
`484`	`487`
`485`	`488`	`static void OnComplianceTxNextPacketTimerEvent( void* context )`