Skip to content

Fix nasa#148, DS App strcpy does not check bound.#149

Open
elizabash wants to merge 1 commit into
devfrom
fix-148-ds-use-of-strcpy
Open

Fix nasa#148, DS App strcpy does not check bound.#149
elizabash wants to merge 1 commit into
devfrom
fix-148-ds-use-of-strcpy

Conversation

@elizabash
Copy link
Copy Markdown

@elizabash elizabash commented May 26, 2026

Checklist (Please check before submitting)

Describe the contribution
Static analyzer via CodeSonar found a usage of 'strcpy' the DS App. Strcpy() does not check bounds and should be replaced with something that does like strncpy(). strncpy(dest, src, sizeof(dest));
Fixes #148

Testing performed

  1. make distclean
  2. make native_std.prep
  3. make native_std.install
  4. Note: I still can't log into CodeSonar to check the static analysis results. It would be great if a reviewer could check what CodeSonar has to say.

@elizabash elizabash self-assigned this May 26, 2026
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.

DS App: Use of strcpy

2 participants