Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #374: CFE_ES_GetTaskInfo must be passed a task ID #375

Merged
merged 1 commit into from
Nov 7, 2019

Conversation

jphickey
Copy link
Contributor

Fix 2 areas where the CFE_ES_GetTaskInfo() function could
have been passed an App ID rather than a task ID, which
would create an incorrect result and generate an error in the syslog

Testing performed
Steps taken to test the contribution:

  • Set EVS default configuration to enable all events (including DEBUG) by default
  • Built all for pc-linux from a clean slate
  • Run CFE core and confirm all startup OK
  • Externally Issue StopApp command for "SAMPLE_APP"
  • Confirmed that all debug events are correct
  • Confirmed that the original error is not generated
  • Externally Issue Shell command for "ES_ListTasks"
  • Confirmed that no bad task ID errors are shown
  • Confirmed that the file output is correct

Expected behavior changes
The invalid task errors shown in the logs prior to this change are NOT generated after this change.

System(s) tested on:
Ubuntu 18.04.2 LTS 64-bit

Contributor Info
Joseph Hickey, Vantage Systems, Inc.

Community contributors
You must attach a signed CLA (required for acceptance) or reference one already submitted

Fix 2 areas where the CFE_ES_GetTaskInfo() function could
have been passed an App ID rather than a task ID, which
would create an incorrect result.
@jphickey jphickey requested a review from skliper October 12, 2019 00:57
@jphickey
Copy link
Contributor Author

Additional notes:

Before this change, with full debug events enabled, issuing the StopApp command for SAMPLE_APP generated these events:

EVS Port1 42/1/CFE_SB 48: Subscription Removed:Msg 0x127 on pipe 5,app CFE_ES
EVS Port1 42/1/CFE_SB 48: Subscription Removed:Msg 0x129 on pipe 5,app CFE_ES
1980-012-14:15:42.50072 CFE_ES_GetTaskInfo: Task ID Not Valid: 5
EVS Port1 42/1/CFE_SB 47: Pipe Deleted:id 5,owner Unknown
EVS Port1 42/1/CFE_ES 14: Exit Application SAMPLE_APP on Error Completed.

After this change, the same command generates these events:

EVS Port1 42/1/CFE_SB 48: Subscription Removed:Msg 0x127 on pipe 5,app CFE_ES
EVS Port1 42/1/CFE_SB 48: Subscription Removed:Msg 0x129 on pipe 5,app CFE_ES
EVS Port1 42/1/CFE_SB 47: Pipe Deleted:id 5,owner SAMPLE_APP
EVS Port1 42/1/CFE_ES 14: Exit Application SAMPLE_APP on Error Completed.

@skliper skliper added this to the 6.8.0 milestone Oct 13, 2019
@stashakk
Copy link

MSFC approved

@CDKnightNASA
Copy link
Contributor

ARC approved

@skliper skliper added the CCB:Approved Indicates code review and approval by community CCB label Oct 23, 2019
@skliper
Copy link
Contributor

skliper commented Oct 23, 2019

CCB 20191023 - reviewed and approved

@skliper
Copy link
Contributor

skliper commented Oct 30, 2019

Fixes #374, wasn't listed in pull request so didn't originally show up as a link.

skliper added a commit that referenced this pull request Oct 31, 2019
Fixes #361, #373, #374, #381
Code reviewed and approved at 20191023 and 30 CCBs
@skliper skliper changed the base branch from master to ic-20191030 November 7, 2019 21:32
@skliper skliper merged commit e9f7382 into nasa:ic-20191030 Nov 7, 2019
@jphickey jphickey deleted the fix-374-deleteapp-error branch October 14, 2020 03:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CCB:Approved Indicates code review and approval by community CCB
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants