-
Notifications
You must be signed in to change notification settings - Fork 1.9k
CPP: Additional change notes. #289
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
Conversation
|
Fixed merge conflict and added a change note for #264 (external contribution). |
change-notes/1.19/analysis-cpp.md
Outdated
| | **Query** | **Tags** | **Purpose** | | ||
| |-----------------------------|-----------|--------------------------------------------------------------------| | ||
| | *@name of query (Query ID)* | *Tags* |*Aim of the new query and whether it is enabled by default or not* | | ||
| | Cast from char* to wchar_t* | security, external/cwe/cwe-704, external/microsoft/c/c6276 | Detects potentially dangerous casts from char* to wchar_t*. Enabled by default on LGTM. | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you think we want those "external/microsoft" tags in this file? I left them out of #235 because I thought they were of too narrow interest and too numerous (one query had 5 of them).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I don't think it's worth mentioning here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
Fix 'unused borrow that must be used' warnings.
Fix (generic) type access extraction for lambdas
Change notes for: