Skip to content

Fix #780, restrict permissions on file create#827

Merged
astrogeco merged 1 commit intonasa:integration-candidatefrom
zanzaben:fix780_restrict_file_create
Mar 1, 2021
Merged

Fix #780, restrict permissions on file create#827
astrogeco merged 1 commit intonasa:integration-candidatefrom
zanzaben:fix780_restrict_file_create

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 force-pushed the fix780_restrict_file_create branch from 9e64376 to 32a5f5f Compare February 22, 2021 19:20
@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