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

MCP750 needs to set context size on exception handling #170

Closed
jphickey opened this issue Jun 16, 2020 · 0 comments · Fixed by #173 or #175
Closed

MCP750 needs to set context size on exception handling #170

jphickey opened this issue Jun 16, 2020 · 0 comments · Fixed by #173 or #175
Assignees
Labels
bug Something isn't working
Milestone

Comments

@jphickey
Copy link
Contributor

Describe the bug
MCP750 is not storing the context size, which results in the size being 0 and therefore no extended data is saved to the log file even though its in memory.

To Reproduce
Found in unit testing with new PSP coverage tests (not yet merged)

Expected behavior
The CFE_PSP_ExceptionHook function should set the context_size field to the amount of data it has actually stored in the buffer. This is then used later if a file dump is requested to write the exception data to a file.

System observed on:
Ubuntu 20.04 running PSP coverage tests.

Reporter Info
Joseph Hickey, Vantage Systems, Inc.

@jphickey jphickey added the bug Something isn't working label Jun 16, 2020
@jphickey jphickey self-assigned this Jun 16, 2020
jphickey added a commit to jphickey/PSP that referenced this issue Jun 16, 2020
Store the size of the stored data into the exception record on
mcp750-vxworks platform.
jphickey added a commit to jphickey/PSP that referenced this issue Jun 16, 2020
Store the size of the stored data into the exception record on
mcp750-vxworks platform.
astrogeco added a commit that referenced this issue Jun 17, 2020
@skliper skliper added this to the 1.5.0 milestone Jun 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants