bugprone-misplaced-pointer-arithmetic-in-alloc.cpp 2.7 KB
// RUN: %check_clang_tidy %s bugprone-misplaced-pointer-arithmetic-in-alloc %t

class C {
  int num;
public:
  explicit C(int n) : num(n) {}
};

void bad_new(int n, int m) {
  C *p = new C(n) + 10;
  // CHECK-MESSAGES: :[[@LINE-1]]:10: warning: arithmetic operation is applied to the result of operator new() instead of its size-like argument
  // CHECK-FIXES: C *p = new C(n + 10);

  p = new C(n) - 10;
  // CHECK-MESSAGES: :[[@LINE-1]]:7: warning: arithmetic operation is applied to the result of operator new() instead of its size-like argument
  // CHECK-FIXES: p = new C(n - 10);

  p = new C(n) + m;
  // CHECK-MESSAGES: :[[@LINE-1]]:7: warning: arithmetic operation is applied to the result of operator new() instead of its size-like argument
  // CHECK-FIXES: p = new C(n + m);

  p = new C(n) - (m + 10);
  // CHECK-MESSAGES: :[[@LINE-1]]:7: warning: arithmetic operation is applied to the result of operator new() instead of its size-like argument
  // CHECK-FIXES: p = new C(n - (m + 10));

  p = new C(n) - m + 10;
  // CHECK-MESSAGES: :[[@LINE-1]]:7: warning: arithmetic operation is applied to the result of operator new() instead of its size-like argument
  // CHECK-FIXES: p = new C(n - m) + 10;
  // FIXME: Should be p = new C(n - m + 10);
}

void bad_new_array(int n, int m) {
  char *p = new char[n] + 10;
  // CHECK-MESSAGES: :[[@LINE-1]]:13: warning: arithmetic operation is applied to the result of operator new[]() instead of its size-like argument
  // CHECK-FIXES: char *p = new char[n + 10];

  p = new char[n] - 10;
  // CHECK-MESSAGES: :[[@LINE-1]]:7: warning: arithmetic operation is applied to the result of operator new[]() instead of its size-like argument
  // CHECK-FIXES: p = new char[n - 10];

  p = new char[n] + m;
  // CHECK-MESSAGES: :[[@LINE-1]]:7: warning: arithmetic operation is applied to the result of operator new[]() instead of its size-like argument
  // CHECK-FIXES: p = new char[n + m];

  p = new char[n] - (m + 10);
  // CHECK-MESSAGES: :[[@LINE-1]]:7: warning: arithmetic operation is applied to the result of operator new[]() instead of its size-like argument
  // CHECK-FIXES: p = new char[n - (m + 10)];

  p = new char[n] - m + 10;
  // CHECK-MESSAGES: :[[@LINE-1]]:7: warning: arithmetic operation is applied to the result of operator new[]() instead of its size-like argument
  // CHECK-FIXES: p = new char[n - m] + 10;
  // FIXME: should be p = new char[n - m + 10];
}

namespace std {
typedef decltype(sizeof(void*)) size_t;
}

void* operator new(std::size_t, void*);

void placement_new_ptr(void *buf, C *old) {
  C **p = new (buf) C*(old) + 1;
  // CHECK-MESSAGES-NOT: :[[@LINE-1]]:11: warning: arithmetic operation is applied to the result of operator new() instead of its size-like argument
}