Skip to content

Fix #761, Terminate string in TestReadWriteLseek#763

Merged
astrogeco merged 1 commit intonasa:integration-candidatefrom
skliper:fix761-ut_file_null
Jan 21, 2021
Merged

Fix #761, Terminate string in TestReadWriteLseek#763
astrogeco merged 1 commit intonasa:integration-candidatefrom
skliper:fix761-ut_file_null

Conversation

@skliper
Copy link
Contributor

@skliper skliper commented Jan 15, 2021

Describe the contribution
Fix #761, Terminate string in TestReadWriteLseek

Testing performed
Initialized the character arrays to known values, observed missing termination in report, fixed w/ this comment and observed the garbage no longer being reported.

Expected behavior changes
No impact to main code, UT only (and just the report, test passes either way).

System(s) tested on

  • cFS Dev Server
  • OS: Ubuntu 18.04
  • Versions: Bundle main + this commit

Additional context
None

Third party code
None

Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

@skliper skliper added this to the 6.0.0 milestone Jan 15, 2021
@astrogeco astrogeco changed the base branch from main to integration-candidate January 21, 2021 18:11
@astrogeco astrogeco merged commit 0af2a55 into nasa:integration-candidate Jan 21, 2021
@skliper skliper deleted the fix761-ut_file_null branch February 1, 2021 21:35
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.

TestReadWriteLseek has off by one error

2 participants