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 #83, Adds null termination to software bus messaging #89

Merged
merged 1 commit into from
Mar 10, 2023

Conversation

chillfig
Copy link
Contributor

@chillfig chillfig commented Mar 7, 2023

Checklist (Please check before submitting)

Describe the contribution
-Fixes #83

Testing performed
make build & lcov on local dev machine

Expected behavior changes
No expected behavior changes

System(s) tested on
Ubuntu 20.04

Additional context
N/A

Third party code
N/A

Contributor Info - All information REQUIRED for consideration of pull request
Justin Figueroa, Vantage Systems

@chillfig chillfig self-assigned this Mar 7, 2023
@chillfig
Copy link
Contributor Author

chillfig commented Mar 8, 2023

Failing check is due to unexpected force command used in the static analysis workflow yml file.

@dzbaker dzbaker merged commit 10a042f into nasa:main Mar 10, 2023
@dmknutsen dmknutsen added this to the Equuleus milestone May 26, 2023
@chillfig chillfig deleted the Add_null_term branch September 5, 2023 05:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improper Null Termination on Software Bus Message handling
4 participants