[PATCH] D70411: [analyzer][WIP] StrChecker: 31.c

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 20 07:38:21 PST 2019


aaron.ballman added a comment.

In D70411#1750690 <https://reviews.llvm.org/D70411#1750690>, @Charusso wrote:

> Somehow the `MallocBugVisitor` is broken in the wild - it does not catch the allocation, and we need to support a lot more to call it non-alpha (see the `FIXME`s). I do not want to spend time on fixing the visitor, but rather I would make this checker alpha, and move on. After a while when we have enough alarms we could see what could go wrong. The reports are not bad, I tried to make it super false-positive free.
>
> @aaron.ballman the name became `security.cert.str.31.c` and we lack the support to invoke every C checker with `str.*.c` (as I know), but for now it should be fine.


The name seems reasonable enough to me -- it at least has all the relevant information in it, which is a great start.


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70411/new/

https://reviews.llvm.org/D70411





More information about the cfe-commits mailing list