[PATCH] D23375: Add kfree( ) to MallocChecker.cpp

Devin Coughlin via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 29 09:45:03 PDT 2016


dcoughlin added a comment.

In https://reviews.llvm.org/D23375#527291, @andrewmw94 wrote:

> One more thing, obviously it should mimic malloc/free's behavior in complaining about delete/new being used.  Should it also complain about free/malloc being used?  I can't  imagine that would be something people would usually intend to do, but I don't think it's really "incorrect."  Thought?


I'm not familiar with linux kernel programming. Is `free()` available in the kernel? If someone did define their own `free()` and used it with `kmalloc`'d memory, would they consider this a false positive?


https://reviews.llvm.org/D23375





More information about the cfe-commits mailing list