Skip to content

Fix #128, strncpy should result in null term#129

Merged
yammajamma merged 1 commit intonasa:integration-candidatefrom
CDKnightNASA:fix-128-strncpy_null_term
Sep 3, 2020
Merged

Fix #128, strncpy should result in null term#129
yammajamma merged 1 commit intonasa:integration-candidatefrom
CDKnightNASA:fix-128-strncpy_null_term

Conversation

@CDKnightNASA
Copy link
Contributor

Describe the contribution
Closes #128

Reduce the size of the strncpy so that it ensures there's a null byte at the end of the string buffer.

Testing performed
make install, make test

System(s) tested on
Debian 10.5

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

@astrogeco astrogeco requested a review from skliper September 2, 2020 14:17
@astrogeco astrogeco changed the title fix #128 - strncpy should result in null term Fix #128, strncpy should result in null term Sep 2, 2020
@astrogeco
Copy link
Contributor

CCB 2020-09-02 APPROVED, fix parenthesis

@CDKnightNASA CDKnightNASA force-pushed the fix-128-strncpy_null_term branch from da440e7 to 446179d Compare September 2, 2020 17:22
@yammajamma yammajamma added CCB:Approved Indicates approval by CCB IC-20200902 labels Sep 2, 2020
@yammajamma yammajamma changed the base branch from main to integration-candidate September 2, 2020 19:00
@yammajamma yammajamma merged commit ec54b1a into nasa:integration-candidate Sep 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CCB:Approved Indicates approval by CCB

Projects

None yet

Development

Successfully merging this pull request may close these issues.

strncpy possibly not null-terminating

4 participants