Fixup #4626 (Write about new checker in releasenotes) [skip ci] #7166
Conversation
chrchr-github
changed the title
releasenotes.txt
Outdated
| @@ -1,7 +1,7 @@ | |||
| Release Notes for Cppcheck 2.17 | |||
|
|
|||
| New checks: | |||
| - | |||
| - Check null pointer return from memory/resource allocation functions. | |||
There was a problem hiding this comment.
Maybe the new IDs nullPointerArithmeticOutOfMemory, nullPointerOutOfMemory, nullPointerOutOfResources should be mentioned.
There was a problem hiding this comment.
Looks like ctunullpointer reports warnings related to allocation failure under the same ID.
There was a problem hiding this comment.
I can't see that ctunullpointer reports related warnings.
There was a problem hiding this comment.
Here's an example from daca:
flashrom-1.4.0/layout.c:199:23: error: Null pointer dereference: filename [ctunullpointer]
flashrom-1.4.0/layout.c:214:42: note: Assuming allocation function fails
flashrom-1.4.0/layout.c:214:35: note: Calling function sanitise_filename, 1st argument is null
flashrom-1.4.0/layout.c:199:23: note: Dereferencing argument filename that is null
There was a problem hiding this comment.
I have created https://trac.cppcheck.net/ticket/13521 for that issue.
|
We could probably generate a list of added IDs from the error messages output. FYI To prevent the CI from running for the PR the individual commits also need |
No description provided.