Skip to content

Add memory-safety guideline - #4654

Merged
bruntib merged 2 commits into
Ericsson:masterfrom
salticecream:master
Aug 27, 2025
Merged

Add memory-safety guideline#4654
bruntib merged 2 commits into
Ericsson:masterfrom
salticecream:master

Conversation

@salticecream

@salticecream salticecream commented Aug 20, 2025

Copy link
Copy Markdown
Contributor

This PR adds a new guideline, memory-safety, which can be useful for developers who are interested in catching specifically memory-safety bugs.

Furthermore, my suggestion (included in the PR) is to raise unix.MismatchedDeallocator and unix.Malloc to HIGH severity. The former always leads to UB on true positives, and the latter almost always.

@bruntib

bruntib commented Aug 25, 2025

Copy link
Copy Markdown
Contributor

Please, extend this list accordingly:


Also, add the new list item to the selected items' list:
Yes, you're right, this should be added dynamically in the future :)

Also, the severity of some checkers are changed (from medium to high, for example). This breaks some tests: https://github.com/Ericsson/codechecker/actions/runs/17100480035/job/48495383987?pr=4654
I would say that severity change of such important checkers should be done in an independent pull request if we decide to change them.

Thank you!

@dkrupp dkrupp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bruntib
bruntib merged commit cb4f5ac into Ericsson:master Aug 27, 2025
8 of 10 checks passed
bruntib pushed a commit that referenced this pull request Sep 19, 2025
* add memory-safety guideline

* add some Clang warnings, revert severity changes, add stats
@bruntib bruntib added this to the release 6.26.2 milestone Oct 14, 2025
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.

3 participants