Fix #121, Add check for UsedFlag and sets APID to CFE_TBL_NOT_OWNED

[dmknutsen]

Describe the contribution
Updates CFE_TBL_CleanUpApp such that it now checks the 'used flag' prior to calling CFE_TBL_RemoveAccessLink for a given TblHandle. Also sets the AppId to CFE_TBL_NOT_OWNED after removing the access descriptor link from linked list.

Fix #121

Testing performed:

1. Modified cfe_es_runloop such that if AppSate == cfe_es_appstate_waiting, then it will not return false. This was done to allow the restart command to work.
2. Created a test app that uses tables.
3. Modified table Noop command such that it prints out Table Access Descriptor data when received.
4. Ran the following test - Note that I sent a noop after each step to review data (test results attached)
   a. Started CFE
   b. Deleted Test App
   c. Restarted Sample App
   d. Restarted Sample App

Ran test twice - Once to ensure I could recreate the error. Once to ensure changes corrected error.

5. Ran unit tests

Expected behavior changes
Will no longer receive 'CFE_TBL:RemoveAccessLink-PutPoolBuf[0] Fail' error.

System(s) tested on
Oracle VM VirtualBox
OS: ubuntu-19.10
Versions: 6.7.12.0, OSAL 5.0.9.0, PSP 1.4.7.0

Contributor Info
Dan Knutsen
NASA/Goddard

issue_121_w_Fix.txt
issue_121_wo_Fix.txt

[astrogeco]

CCB 20200422 - Approved

[jphickey]

This pull request also breaking the current integration candidate:

/home/joe/code/cfecfs/github/cfe/fsw/cfe-core/src/tbl/cfe_tbl_internal.c:1394:99: error: ‘TRUE’ undeclared (first use in this function)
 1394 |         if (CFE_TBL_TaskData.Handles[i].AppId == AppId && CFE_TBL_TaskData.Handles[i].UsedFlag == TRUE)
      |                                                                                                   ^~~~

Should be using true not TRUE here....