Ericsson / clang

Cross Translation Unit analysis capability for Clang Static Analyzer. (Fork of official clang at http://llvm.org/git/clang)
http://clang.llvm.org/
Other
15 stars 10 forks source link

Reapply "[analyzer] Introduce a simplified API for adding custom path notes." #675

Closed baloghadamsoftware closed 5 years ago

baloghadamsoftware commented 5 years ago

This reapplies commit r357323, fixing memory leak found by LSan.

Differential Revision: https://reviews.llvm.org/D58367

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@358781 91177308-0d34-0410-b5e6-96231b3b80d8

baloghadamsoftware commented 5 years ago

I could not find a proper solution for the memory leak in passing string between the the statistics-based checkers and their bug reporter visitor so I cherry-picked the new method from the upstream clang. This seems to work without problems.