[PATCH] D154603: [analyzer][clangsa] Add new option to alpha.security.cert.InvalidPtrChecker

Wed Aug 23 06:05:03 PDT 2023

gamesh411 added a comment.

In D154603#4580609 <https://reviews.llvm.org/D154603#4580609>, @steakhal wrote:

> I'm sorry starting the review of this one only now, but I'm quite booked.
> Is it still relevant? If so, I'll continue.

Yes thanks for the effort!
I would like to go through with this option, and then I would like to fix the following issues with this checker as well:

- the previous function call notes could be more streamlined
- the interesting notes are also shown, when another checker hits those nodes with its report (for example taint checker giving a warning for getenv would also trigger the display of the 'previous function call was here' note here), this I would like to filter with bug category filters
- try to consolidate the multiple warnings coming from this checkers checkLocation callback

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154603