Merge pull request #685 from jphickey/fix-684-cleanup-reorder

Fix #684, order of CleanupTaskResources
nasa · May 19, 2020 · df31b09 · df31b09
2 parents 28faa4c + 8dad536
commit df31b09
Showing 1 changed file with 13 additions and 13 deletions.
diff --git a/fsw/cfe-core/src/es/cfe_es_apps.c b/fsw/cfe-core/src/es/cfe_es_apps.c
@@ -1114,24 +1114,12 @@ int32 CFE_ES_CleanUpApp(uint32 AppId)
    */
    MainTaskId = CFE_ES_Global.AppTable[AppId].TaskInfo.MainTaskId;
 
-   /*
-   ** Delete all of the OS resources, close files, and delete the main task
-   */
-   Status = CFE_ES_CleanupTaskResources(MainTaskId);
-   if ( Status != CFE_SUCCESS )
-   {
-      CFE_ES_SysLogWrite_Unsync("CFE_ES_CleanUpApp: CleanUpTaskResources for Task ID:%d returned Error: 0x%08X\n",
-                               (int)MainTaskId, (unsigned int)Status);
-      ReturnCode = CFE_ES_APP_CLEANUP_ERR;
-
-   }
-
    /*
    ** Delete any child tasks associated with this app
    */
    for ( i = 0; i < OS_MAX_TASKS; i++ )
    {
-      /* delete only CHILD tasks - not the MainTaskId, which is already deleted (above) */
+      /* delete only CHILD tasks - not the MainTaskId, which will be deleted later (below) */
       if ((CFE_ES_Global.TaskTable[i].RecordUsed == true) &&
           (CFE_ES_Global.TaskTable[i].AppId == AppId) &&
           (CFE_ES_Global.TaskTable[i].TaskId != MainTaskId))
@@ -1146,6 +1134,18 @@ int32 CFE_ES_CleanUpApp(uint32 AppId)
       } /* end if */
    } /* end for */
 
+   /*
+   ** Delete all of the OS resources, close files, and delete the main task
+   */
+   Status = CFE_ES_CleanupTaskResources(MainTaskId);
+   if ( Status != CFE_SUCCESS )
+   {
+      CFE_ES_SysLogWrite_Unsync("CFE_ES_CleanUpApp: CleanUpTaskResources for Task ID:%d returned Error: 0x%08X\n",
+                               (int)MainTaskId, (unsigned int)Status);
+      ReturnCode = CFE_ES_APP_CLEANUP_ERR;
+
+   }
+
    /*
    ** Remove the app from the AppTable
    */