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 #156, termination on strncpy #157

Merged
merged 1 commit into from
May 8, 2020

Conversation

jphickey
Copy link
Contributor

Describe the contribution
Fix possible non-termination of strings in command line option parsing.
These generated warnings in GCC9.

Fixes #156

Testing performed
Build code with default config, SIMULATION=native BUILDTYPE=release on GCC 9.3.0.
Confirm successful build with no warning.

Expected behavior changes
No impact to behavior

System(s) tested on
Ubuntu 20.04 LTS 64 bit

Contributor Info - All information REQUIRED for consideration of pull request
Joseph Hickey, Vantage Systems, Inc.

Fix possible non-termination of strings in command line option parsing.
These generated warnings in GCC9.
@jphickey jphickey added bug Something isn't working CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels Apr 27, 2020
@astrogeco
Copy link
Contributor

CCB 2020-04-29 : APPROVED

@astrogeco astrogeco added CCB-20200429 CCB:Approved Indicates Approval by CCB and removed CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels May 5, 2020
@astrogeco astrogeco changed the base branch from master to integration-candidate May 8, 2020 19:18
@astrogeco astrogeco merged commit d1872ed into nasa:integration-candidate May 8, 2020
@jphickey jphickey deleted the fix-156-stringops branch May 14, 2020 14:56
@skliper skliper added this to the 1.5.0 milestone Jun 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working CCB:Approved Indicates Approval by CCB
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Many string truncation warnings with newer GCC
3 participants