]> granicus.if.org Git - clang/commitdiff
[analyzer] Makes NewDeleteLeaks checker work independently from NewDelete.
authorAnton Yartsev <anton.yartsev@gmail.com>
Fri, 12 Apr 2013 20:48:49 +0000 (20:48 +0000)
committerAnton Yartsev <anton.yartsev@gmail.com>
Fri, 12 Apr 2013 20:48:49 +0000 (20:48 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@179410 91177308-0d34-0410-b5e6-96231b3b80d8

lib/StaticAnalyzer/Checkers/MallocChecker.cpp
test/Analysis/NewDeleteLeaks-checker-test.cpp [new file with mode: 0644]

index e19e3f72c7f0ae383167b9e31c1d6f249b792967..246dc1588b07f90f96ac9291f8f37508d61d2bed 100644 (file)
@@ -1091,7 +1091,7 @@ bool MallocChecker::isTrackedByCurrentChecker(AllocationFamily Family) const {
   }
   case AF_CXXNew:
   case AF_CXXNewArray: {
-    if (!Filter.CNewDeleteChecker)
+    if (!Filter.CNewDeleteChecker && !Filter.CNewDeleteLeaksChecker)
       return false;
     return true;
   }
diff --git a/test/Analysis/NewDeleteLeaks-checker-test.cpp b/test/Analysis/NewDeleteLeaks-checker-test.cpp
new file mode 100644 (file)
index 0000000..93707ec
--- /dev/null
@@ -0,0 +1,28 @@
+// RUN: %clang_cc1 -analyze -analyzer-checker=core,alpha.cplusplus.NewDeleteLeaks -std=c++11 -fblocks -verify %s
+#include "Inputs/system-header-simulator-cxx.h"
+
+//----- Standard non-placement operators
+void testGlobalOpNew() {
+  void *p = operator new(0);
+} // expected-warning{{Potential leak of memory pointed to by 'p'}}
+
+void testGlobalOpNewArray() {
+  void *p = operator new[](0);
+} // expected-warning{{Potential leak of memory pointed to by 'p'}}
+
+void testGlobalNewExpr() {
+  int *p = new int;
+} // expected-warning{{Potential leak of memory pointed to by 'p'}}
+
+void testGlobalNewExprArray() {
+  int *p = new int[0];
+} // expected-warning{{Potential leak of memory pointed to by 'p'}}
+
+//----- Standard nothrow placement operators
+void testGlobalNoThrowPlacementOpNewBeforeOverload() {
+  void *p = operator new(0, std::nothrow);
+} // expected-warning{{Potential leak of memory pointed to by 'p'}}
+
+void testGlobalNoThrowPlacementExprNewBeforeOverload() {
+  int *p = new(std::nothrow) int;
+} // expected-warning{{Potential leak of memory pointed to by 'p'}}