-
Notifications
You must be signed in to change notification settings - Fork 208
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
Ensure that return codes from PSP functions are checked #60
Comments
Imported from trac issue 29. Created by jphickey on 2015-01-22T12:40:14, last modified: 2019-03-05T14:57:55 |
Trac comment by jphickey on 2015-01-26 16:25:58: Branch "trac-29-psp_retcodes" pushed -- commit [changeset:2fdeec90] This ensures that the return codes of PSP functions are checked per the above description. |
Trac comment by jphickey on 2015-04-06 11:31:07: This is ready for review/merge |
Trac comment by sstrege on 2015-04-06 13:07:56: Concur with changes/merge |
Trac comment by acudmore on 2015-04-07 12:40:40: Concur |
Trac comment by glimes on 2015-04-13 15:21:56: Part of integration candidate 2015-03-10, |
Trac comment by glimes on 2016-02-25 10:17:32: these will be fixed in CFE 6.5 |
Trac comment by jhageman on 2019-03-05 14:57:55: Milestone renamed |
In some circumstances the return code from PSP functions (e.g. CFE_PSP_GetResetArea) is either not checked or incorrectly checked.
Wherever the return code is checked, instead of specifically checking for an error code like so:
{{{
if (Status == CFE_PSP_ERROR)
{
}
}}}
The code should check for non-success instead:
{{{
if (Status != CFE_PSP_SUCCESS)
{
}
}}}
This will make it more future-proof to PSP implementations that might return more specific errors.
The text was updated successfully, but these errors were encountered: