Not to use object modified in the loop body (#861)
Co-authored-by: Ubuntu <ubuntu@ip-172-31-34-245.ap-northeast-1.compute.internal>
Co-authored-by: Rahul Kar <118818625+kar-rahul-aws@users.noreply.github.com>
Co-authored-by: Gaurav-Aggarwal-AWS <33462878+aggarg@users.noreply.github.com>
diff --git a/tasks.c b/tasks.c
index 7aa93f7..0907d35 100644
--- a/tasks.c
+++ b/tasks.c
@@ -7231,7 +7231,7 @@
uxArraySize = uxTaskGetSystemState( pxTaskStatusArray, uxArraySize, NULL );
/* Create a human readable table from the binary data. */
- for( x = 0; ( x < uxArraySize ) && ( xOutputBufferFull == pdFALSE ); x++ )
+ for( x = 0; x < uxArraySize; x++ )
{
switch( pxTaskStatusArray[ x ].eCurrentState )
{
@@ -7310,6 +7310,11 @@
{
xOutputBufferFull = pdTRUE;
}
+
+ if( xOutputBufferFull == pdTRUE )
+ {
+ break;
+ }
}
/* Free the array again. NOTE! If configSUPPORT_DYNAMIC_ALLOCATION
@@ -7391,7 +7396,7 @@
if( ulTotalTime > 0UL )
{
/* Create a human readable table from the binary data. */
- for( x = 0; ( x < uxArraySize ) && ( xOutputBufferFull == pdFALSE ); x++ )
+ for( x = 0; x < uxArraySize; x++ )
{
/* What percentage of the total run time has the task used?
* This will always be rounded down to the nearest integer.
@@ -7472,6 +7477,11 @@
{
xOutputBufferFull = pdTRUE;
}
+
+ if( xOutputBufferFull == pdTRUE )
+ {
+ break;
+ }
}
}
else