FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

C++: Add `cpp/non-constant-format` test by jketema · Pull Request #14021 · github/codeql · GitHub

/ codeql Public

C++: Add cpp/non-constant-format test - #14021

Merged
jketema merged 2 commits into
github:mainfrom
jketema:non-const-format-test
Aug 22, 2023
Merged

C++: Add cpp/non-constant-format test#14021
jketema merged 2 commits into
github:mainfrom
jketema:non-const-format-test

Conversation

jketema commented Aug 22, 2023

Copy link
Copy Markdown
Contributor

No description provided.

jketema requested a review from a team as a code owner August 22, 2023 14:24
github-actions Bot added the C++ label Aug 22, 2023
MathiasVP previously approved these changes Aug 22, 2023

jketema commented Aug 22, 2023

Copy link
Copy Markdown
Contributor Author

This change adds a safer function to demonstrate proper usage of formatted strings in C++ and avoid buffer overflows and vulnerabilities.

  • cpp/ql/test/query-tests/Likely Bugs/Format/NonConstantFormat/test.cpp: Added a safer function fmt_with_assignment() to demonstrate proper usage of formatted strings in C++ and avoid buffer overflows and vulnerabilities. (cpp/ql/test/query-tests/Likely Bugs/Format/NonConstantFormat/test.cppR159-R165)

jketema merged commit cf53956 into github:main Aug 22, 2023
jketema deleted the non-const-format-test branch August 22, 2023 15:38

jketema commented Aug 22, 2023

Copy link
Copy Markdown
Contributor Author

Note to anyone who might read the previous comment later. This was an attempt at an autogenerated summary.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL