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 #780, restrict permissions on file create #827

Merged

Conversation

zanzaben
Copy link
Contributor

@zanzaben zanzaben commented Feb 22, 2021

Describe the contribution
Fixes #780
adds restriction check to file open.

Testing performed
Build and run unit test

Expected behavior changes
No impact to behavior

System(s) tested on
Ubuntu 20.04

Contributor Info - All information REQUIRED for consideration of pull request
Alex Campbell GSFC

@zanzaben zanzaben added CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) and removed CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels Feb 22, 2021
@zanzaben zanzaben added the CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) label Feb 23, 2021
@astrogeco astrogeco added CCB:2021-02-24 and removed CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels Feb 24, 2021
@astrogeco
Copy link
Contributor

astrogeco commented Feb 24, 2021

CCB 2021-02-24 APPROVED

  • doesn't touch flight code

@astrogeco astrogeco changed the base branch from main to integration-candidate March 1, 2021 22:16
@astrogeco astrogeco merged commit 9725f44 into nasa:integration-candidate Mar 1, 2021
@skliper skliper added this to the 6.0.0 milestone Sep 24, 2021
jphickey pushed a commit to jphickey/osal that referenced this pull request Aug 10, 2022
Removed CFE_PLATFORM_CPU_ID, CFE_PLATFORM_CPU_NAME,
and CFE_MISSION_SPACECRAFT_ID.  Use CFE_PSP_GetProcessorId(),
CFE_PSP_GetProcessorName(), CFE_PSP_GetSpacecraftId().
jphickey pushed a commit to jphickey/osal that referenced this pull request Aug 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

File created without restricting permissions in unit test tools
3 participants