[PATCH] [analyzer][Review request] Partial fix for PR19102.

Jordan Rose jordan_rose at apple.com
Tue Aug 5 10:20:44 PDT 2014


It looks like many of them are related to mangling the address of a pointer with other bits in an intptr_t, which is a known area of concern. I'm actually very happy that we're not seeing hundreds of false positives from the "parameter takes ownership" pattern, so this is encouraging. And as far as I can tell the leak in ScalarEvolution is real!

Thanks for running this, Anton; I'm much less worried about turning this checker on now.
Jordan

On Aug 4, 2014, at 12:34 , Anton Yartsev <anton.yartsev at gmail.com> wrote:

> Attached are leak reports generated by NewDeleteLeaks checker ran over the LLVM codebase with -analyzer-opt-analyze-headers option turned on. Haven't addressed them yet. Looking at suspicious leak reports generated from headers (e.g. reports from TinyPtrVector.h).
> 
> -- 
> Anton
> 
> <NewDeleteLeaks_reports.zip>




More information about the cfe-commits mailing list