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 #730 - adds null to FILEDEFs #738

Merged

Conversation

CDKnightNASA
Copy link
Contributor

Describe the contribution
Fix #730 - Adds "\0" in the CFE_TBL_FILEDEF() macro so that if a string is exactly the size of the field, it will produce a compiler error.

Contributor Info - All information REQUIRED for consideration of pull request
[email protected]

@CDKnightNASA CDKnightNASA added the CCB:Ready Ready for discussion at the Configuration Control Board (CCB) label Jun 11, 2020
@CDKnightNASA CDKnightNASA self-assigned this Jun 11, 2020
@astrogeco
Copy link
Contributor

CCB 2020-06-17: APPROVED

@astrogeco astrogeco added CCB-20200617 CCB:Approved Indicates code review and approval by community CCB and removed CCB:Ready Ready for discussion at the Configuration Control Board (CCB) labels Jun 17, 2020
@astrogeco astrogeco changed the base branch from master to integration-candidate June 23, 2020 19:09
@astrogeco astrogeco merged commit dcb00a4 into nasa:integration-candidate Jun 23, 2020
@skliper skliper added this to the 6.8.0 milestone Aug 21, 2020
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.

low CFE_MISSION_MAX_FILE_LEN and CFE_TBL_FILEDEF(..., "sample_app_table.tbl") causes silent non-truncation
3 participants