commit 953c1ee6cbf460092826bab772e4119c6d9e51e8
author Gaurav-Aggarwal-AWS <33462878+aggarg@users.noreply.github.com> Thu May 25 16:33:10 2023 +0530
committer GitHub <noreply@github.com> Thu May 25 16:33:10 2023 +0530
tree 72bc848598d75c537bede0511d0acb2a5a1e26aa
parent 97434a4e0cfcf8dbd5cceeaf68c3f37121b03996

Fix eTaskGetState for pending ready tasks (#679)

This commit fixes eTaskGetState so that eReady is returned for pending ready
tasks.

Co-authored-by: Darian Leung <darian@espressif.com>

tasks.c

diff --git a/tasks.c b/tasks.c
index 59f9622..97539d7 100644
--- a/tasks.c
+++ b/tasks.c
@@ -1351,6 +1351,7 @@
     {
         eTaskState eReturn;
         List_t const * pxStateList;
+        List_t const * pxEventList;
         List_t const * pxDelayedList;
         List_t const * pxOverflowedDelayedList;
         const TCB_t * const pxTCB = xTask;
@@ -1367,12 +1368,20 @@
             taskENTER_CRITICAL();
             {
                 pxStateList = listLIST_ITEM_CONTAINER( &( pxTCB->xStateListItem ) );
+                pxEventList = listLIST_ITEM_CONTAINER( &( pxTCB->xEventListItem ) );
                 pxDelayedList = pxDelayedTaskList;
                 pxOverflowedDelayedList = pxOverflowDelayedTaskList;
             }
             taskEXIT_CRITICAL();
 
-            if( ( pxStateList == pxDelayedList ) || ( pxStateList == pxOverflowedDelayedList ) )
+            if( pxEventList == &xPendingReadyList )
+            {
+                /* The task has been placed on the pending ready list, so its
+                 * state is eReady regardless of what list the task's state list
+                 * item is currently placed on. */
+                eReturn = eReady;
+            }
+            else if( ( pxStateList == pxDelayedList ) || ( pxStateList == pxOverflowedDelayedList ) )
             {
                 /* The task being queried is referenced from one of the Blocked
                  * lists. */