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

Set RunStatus to CFE_ES_RunStatus_APP_ERROR if initialization fails #187

Closed
2 tasks done
thnkslprpt opened this issue Jan 17, 2024 · 0 comments · Fixed by #188
Closed
2 tasks done

Set RunStatus to CFE_ES_RunStatus_APP_ERROR if initialization fails #187

thnkslprpt opened this issue Jan 17, 2024 · 0 comments · Fixed by #188
Labels
enhancement New feature or request

Comments

@thnkslprpt
Copy link
Contributor

Checklist

  • I reviewed the Contributing Guide.
  • I performed a cursory search to see if the bug report is relevant, not redundant, nor in conflict with other tickets.

Describe the bug
Would be cleaner to set RunStatus to CFE_ES_RunStatus_APP_ERROR if initialization fails, rather than just a simple return:

Code snips

status = TO_LAB_init();
if (status != CFE_SUCCESS)
{
return;
}

Reporter Info
Avi Weiss   @thnkslprpt

@chillfig chillfig added the enhancement New feature or request label Apr 11, 2024
dzbaker added a commit that referenced this issue Apr 16, 2024
…ERROR-if-init-fails

Fix #187, Zero-out global data during init + set RunStatus to APP_ERROR if init fails
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
2 participants