Dominic Chen | 184c624 | 2017-03-03 18:02:02 +0000 | [diff] [blame] | 1 | // RUN: %clang_analyze_cc1 -analyzer-checker=core,cplusplus.NewDelete -std=c++11 -fblocks -verify %s |
| 2 | // RUN: %clang_analyze_cc1 -analyzer-checker=core,cplusplus.NewDeleteLeaks -DLEAKS -std=c++11 -fblocks -verify %s |
Artem Dergachev | 1084de5 | 2018-01-17 22:58:35 +0000 | [diff] [blame^] | 3 | // RUN: %clang_analyze_cc1 -analyzer-checker=core,cplusplus.NewDelete -std=c++11 -fblocks -analyzer-config c++-allocator-inlining=true -verify %s |
| 4 | // RUN: %clang_analyze_cc1 -analyzer-checker=core,cplusplus.NewDeleteLeaks -DLEAKS -std=c++11 -fblocks -analyzer-config c++-allocator-inlining=true -verify %s |
Anton Yartsev | 13df036 | 2013-03-25 01:35:45 +0000 | [diff] [blame] | 5 | #include "Inputs/system-header-simulator-cxx.h" |
Anton Yartsev | 13df036 | 2013-03-25 01:35:45 +0000 | [diff] [blame] | 6 | |
| 7 | typedef __typeof__(sizeof(int)) size_t; |
| 8 | extern "C" void *malloc(size_t); |
Jordan Rose | 867b185e | 2013-08-09 00:55:47 +0000 | [diff] [blame] | 9 | extern "C" void free (void* ptr); |
Anton Yartsev | 13df036 | 2013-03-25 01:35:45 +0000 | [diff] [blame] | 10 | int *global; |
| 11 | |
| 12 | //------------------ |
| 13 | // check for leaks |
| 14 | //------------------ |
| 15 | |
Anton Yartsev | 8b66270 | 2013-03-28 16:10:38 +0000 | [diff] [blame] | 16 | //----- Standard non-placement operators |
| 17 | void testGlobalOpNew() { |
Anton Yartsev | 13df036 | 2013-03-25 01:35:45 +0000 | [diff] [blame] | 18 | void *p = operator new(0); |
Jordan Rose | 2633056 | 2013-04-05 17:55:00 +0000 | [diff] [blame] | 19 | } |
| 20 | #ifdef LEAKS |
Anna Zaks | a1de856 | 2013-04-06 00:41:36 +0000 | [diff] [blame] | 21 | // expected-warning@-2{{Potential leak of memory pointed to by 'p'}} |
Jordan Rose | 2633056 | 2013-04-05 17:55:00 +0000 | [diff] [blame] | 22 | #endif |
Anton Yartsev | 13df036 | 2013-03-25 01:35:45 +0000 | [diff] [blame] | 23 | |
Anton Yartsev | 8b66270 | 2013-03-28 16:10:38 +0000 | [diff] [blame] | 24 | void testGlobalOpNewArray() { |
| 25 | void *p = operator new[](0); |
Jordan Rose | 2633056 | 2013-04-05 17:55:00 +0000 | [diff] [blame] | 26 | } |
| 27 | #ifdef LEAKS |
Anna Zaks | a1de856 | 2013-04-06 00:41:36 +0000 | [diff] [blame] | 28 | // expected-warning@-2{{Potential leak of memory pointed to by 'p'}} |
Jordan Rose | 2633056 | 2013-04-05 17:55:00 +0000 | [diff] [blame] | 29 | #endif |
Anton Yartsev | 8b66270 | 2013-03-28 16:10:38 +0000 | [diff] [blame] | 30 | |
| 31 | void testGlobalNewExpr() { |
Anton Yartsev | 13df036 | 2013-03-25 01:35:45 +0000 | [diff] [blame] | 32 | int *p = new int; |
Jordan Rose | 2633056 | 2013-04-05 17:55:00 +0000 | [diff] [blame] | 33 | } |
| 34 | #ifdef LEAKS |
Anna Zaks | a1de856 | 2013-04-06 00:41:36 +0000 | [diff] [blame] | 35 | // expected-warning@-2{{Potential leak of memory pointed to by 'p'}} |
Jordan Rose | 2633056 | 2013-04-05 17:55:00 +0000 | [diff] [blame] | 36 | #endif |
Anton Yartsev | 13df036 | 2013-03-25 01:35:45 +0000 | [diff] [blame] | 37 | |
Anton Yartsev | 8b66270 | 2013-03-28 16:10:38 +0000 | [diff] [blame] | 38 | void testGlobalNewExprArray() { |
| 39 | int *p = new int[0]; |
Jordan Rose | 2633056 | 2013-04-05 17:55:00 +0000 | [diff] [blame] | 40 | } |
| 41 | #ifdef LEAKS |
Anna Zaks | a1de856 | 2013-04-06 00:41:36 +0000 | [diff] [blame] | 42 | // expected-warning@-2{{Potential leak of memory pointed to by 'p'}} |
Jordan Rose | 2633056 | 2013-04-05 17:55:00 +0000 | [diff] [blame] | 43 | #endif |
Anton Yartsev | 13df036 | 2013-03-25 01:35:45 +0000 | [diff] [blame] | 44 | |
Anton Yartsev | 8b66270 | 2013-03-28 16:10:38 +0000 | [diff] [blame] | 45 | //----- Standard nothrow placement operators |
| 46 | void testGlobalNoThrowPlacementOpNewBeforeOverload() { |
| 47 | void *p = operator new(0, std::nothrow); |
Jordan Rose | 2633056 | 2013-04-05 17:55:00 +0000 | [diff] [blame] | 48 | } |
| 49 | #ifdef LEAKS |
Anna Zaks | a1de856 | 2013-04-06 00:41:36 +0000 | [diff] [blame] | 50 | // expected-warning@-2{{Potential leak of memory pointed to by 'p'}} |
Jordan Rose | 2633056 | 2013-04-05 17:55:00 +0000 | [diff] [blame] | 51 | #endif |
Anton Yartsev | 8b66270 | 2013-03-28 16:10:38 +0000 | [diff] [blame] | 52 | |
| 53 | void testGlobalNoThrowPlacementExprNewBeforeOverload() { |
| 54 | int *p = new(std::nothrow) int; |
Jordan Rose | 2633056 | 2013-04-05 17:55:00 +0000 | [diff] [blame] | 55 | } |
| 56 | #ifdef LEAKS |
Anna Zaks | a1de856 | 2013-04-06 00:41:36 +0000 | [diff] [blame] | 57 | // expected-warning@-2{{Potential leak of memory pointed to by 'p'}} |
Jordan Rose | 2633056 | 2013-04-05 17:55:00 +0000 | [diff] [blame] | 58 | #endif |
Anton Yartsev | 8b66270 | 2013-03-28 16:10:38 +0000 | [diff] [blame] | 59 | |
Anton Yartsev | 8b66270 | 2013-03-28 16:10:38 +0000 | [diff] [blame] | 60 | //----- Standard pointer placement operators |
| 61 | void testGlobalPointerPlacementNew() { |
Anton Yartsev | 13df036 | 2013-03-25 01:35:45 +0000 | [diff] [blame] | 62 | int i; |
Anton Yartsev | 13df036 | 2013-03-25 01:35:45 +0000 | [diff] [blame] | 63 | |
Anton Yartsev | 8b66270 | 2013-03-28 16:10:38 +0000 | [diff] [blame] | 64 | void *p1 = operator new(0, &i); // no warn |
Anton Yartsev | 13df036 | 2013-03-25 01:35:45 +0000 | [diff] [blame] | 65 | |
Anton Yartsev | 8b66270 | 2013-03-28 16:10:38 +0000 | [diff] [blame] | 66 | void *p2 = operator new[](0, &i); // no warn |
Anton Yartsev | 13df036 | 2013-03-25 01:35:45 +0000 | [diff] [blame] | 67 | |
Anton Yartsev | 8b66270 | 2013-03-28 16:10:38 +0000 | [diff] [blame] | 68 | int *p3 = new(&i) int; // no warn |
Anton Yartsev | 13df036 | 2013-03-25 01:35:45 +0000 | [diff] [blame] | 69 | |
Anton Yartsev | 8b66270 | 2013-03-28 16:10:38 +0000 | [diff] [blame] | 70 | int *p4 = new(&i) int[0]; // no warn |
Anton Yartsev | 13df036 | 2013-03-25 01:35:45 +0000 | [diff] [blame] | 71 | } |
| 72 | |
Anton Yartsev | 8b66270 | 2013-03-28 16:10:38 +0000 | [diff] [blame] | 73 | //----- Other cases |
| 74 | void testNewMemoryIsInHeap() { |
| 75 | int *p = new int; |
| 76 | if (global != p) // condition is always true as 'p' wraps a heap region that |
| 77 | // is different from a region wrapped by 'global' |
| 78 | global = p; // pointer escapes |
Anton Yartsev | 13df036 | 2013-03-25 01:35:45 +0000 | [diff] [blame] | 79 | } |
| 80 | |
Anton Yartsev | 8b66270 | 2013-03-28 16:10:38 +0000 | [diff] [blame] | 81 | struct PtrWrapper { |
| 82 | int *x; |
Anton Yartsev | 13df036 | 2013-03-25 01:35:45 +0000 | [diff] [blame] | 83 | |
Anton Yartsev | 8b66270 | 2013-03-28 16:10:38 +0000 | [diff] [blame] | 84 | PtrWrapper(int *input) : x(input) {} |
| 85 | }; |
Anton Yartsev | 13df036 | 2013-03-25 01:35:45 +0000 | [diff] [blame] | 86 | |
Anton Yartsev | 8b66270 | 2013-03-28 16:10:38 +0000 | [diff] [blame] | 87 | void testNewInvalidationPlacement(PtrWrapper *w) { |
| 88 | // Ensure that we don't consider this a leak. |
| 89 | new (w) PtrWrapper(new int); // no warn |
Anton Yartsev | 13df036 | 2013-03-25 01:35:45 +0000 | [diff] [blame] | 90 | } |
| 91 | |
Anton Yartsev | b50f4ba | 2015-04-14 14:18:04 +0000 | [diff] [blame] | 92 | //----------------------------------------- |
| 93 | // check for usage of zero-allocated memory |
| 94 | //----------------------------------------- |
| 95 | |
| 96 | void testUseZeroAlloc1() { |
| 97 | int *p = (int *)operator new(0); |
| 98 | *p = 1; // expected-warning {{Use of zero-allocated memory}} |
| 99 | delete p; |
| 100 | } |
| 101 | |
| 102 | int testUseZeroAlloc2() { |
| 103 | int *p = (int *)operator new[](0); |
| 104 | return p[0]; // expected-warning {{Use of zero-allocated memory}} |
| 105 | delete[] p; |
| 106 | } |
| 107 | |
| 108 | void f(int); |
| 109 | |
| 110 | void testUseZeroAlloc3() { |
| 111 | int *p = new int[0]; |
| 112 | f(*p); // expected-warning {{Use of zero-allocated memory}} |
| 113 | delete[] p; |
| 114 | } |
| 115 | |
Anton Yartsev | 13df036 | 2013-03-25 01:35:45 +0000 | [diff] [blame] | 116 | //--------------- |
| 117 | // other checks |
| 118 | //--------------- |
| 119 | |
Anton Yartsev | cb2ccd6 | 2013-04-10 22:21:41 +0000 | [diff] [blame] | 120 | class SomeClass { |
| 121 | public: |
| 122 | void f(int *p); |
| 123 | }; |
Anton Yartsev | 13df036 | 2013-03-25 01:35:45 +0000 | [diff] [blame] | 124 | |
Anton Yartsev | cb2ccd6 | 2013-04-10 22:21:41 +0000 | [diff] [blame] | 125 | void f(int *p1, int *p2 = 0, int *p3 = 0); |
| 126 | void g(SomeClass &c, ...); |
| 127 | |
| 128 | void testUseFirstArgAfterDelete() { |
Anton Yartsev | 13df036 | 2013-03-25 01:35:45 +0000 | [diff] [blame] | 129 | int *p = new int; |
| 130 | delete p; |
| 131 | f(p); // expected-warning{{Use of memory after it is freed}} |
| 132 | } |
| 133 | |
Anton Yartsev | cb2ccd6 | 2013-04-10 22:21:41 +0000 | [diff] [blame] | 134 | void testUseMiddleArgAfterDelete(int *p) { |
| 135 | delete p; |
| 136 | f(0, p); // expected-warning{{Use of memory after it is freed}} |
| 137 | } |
| 138 | |
| 139 | void testUseLastArgAfterDelete(int *p) { |
| 140 | delete p; |
| 141 | f(0, 0, p); // expected-warning{{Use of memory after it is freed}} |
| 142 | } |
| 143 | |
Anton Yartsev | 8fc29db | 2013-04-10 22:36:16 +0000 | [diff] [blame] | 144 | void testUseSeveralArgsAfterDelete(int *p) { |
| 145 | delete p; |
| 146 | f(p, p, p); // expected-warning{{Use of memory after it is freed}} |
| 147 | } |
| 148 | |
Anton Yartsev | cb2ccd6 | 2013-04-10 22:21:41 +0000 | [diff] [blame] | 149 | void testUseRefArgAfterDelete(SomeClass &c) { |
| 150 | delete &c; |
| 151 | g(c); // expected-warning{{Use of memory after it is freed}} |
| 152 | } |
| 153 | |
| 154 | void testVariadicArgAfterDelete() { |
| 155 | SomeClass c; |
| 156 | int *p = new int; |
| 157 | delete p; |
| 158 | g(c, 0, p); // expected-warning{{Use of memory after it is freed}} |
| 159 | } |
| 160 | |
| 161 | void testUseMethodArgAfterDelete(int *p) { |
| 162 | SomeClass *c = new SomeClass; |
| 163 | delete p; |
| 164 | c->f(p); // expected-warning{{Use of memory after it is freed}} |
| 165 | } |
| 166 | |
| 167 | void testUseThisAfterDelete() { |
| 168 | SomeClass *c = new SomeClass; |
| 169 | delete c; |
| 170 | c->f(0); // expected-warning{{Use of memory after it is freed}} |
| 171 | } |
| 172 | |
Anton Yartsev | 13df036 | 2013-03-25 01:35:45 +0000 | [diff] [blame] | 173 | void testDoubleDelete() { |
| 174 | int *p = new int; |
| 175 | delete p; |
| 176 | delete p; // expected-warning{{Attempt to free released memory}} |
| 177 | } |
| 178 | |
| 179 | void testExprDeleteArg() { |
| 180 | int i; |
Anton Yartsev | 0578959 | 2013-03-28 17:05:19 +0000 | [diff] [blame] | 181 | delete &i; // expected-warning{{Argument to 'delete' is the address of the local variable 'i', which is not memory allocated by 'new'}} |
| 182 | } |
Anton Yartsev | 13df036 | 2013-03-25 01:35:45 +0000 | [diff] [blame] | 183 | |
| 184 | void testExprDeleteArrArg() { |
| 185 | int i; |
Anton Yartsev | 0578959 | 2013-03-28 17:05:19 +0000 | [diff] [blame] | 186 | delete[] &i; // expected-warning{{Argument to 'delete[]' is the address of the local variable 'i', which is not memory allocated by 'new[]'}} |
| 187 | } |
Anton Yartsev | 13df036 | 2013-03-25 01:35:45 +0000 | [diff] [blame] | 188 | |
| 189 | void testAllocDeallocNames() { |
| 190 | int *p = new(std::nothrow) int[1]; |
Anton Yartsev | 0578959 | 2013-03-28 17:05:19 +0000 | [diff] [blame] | 191 | delete[] (++p); // expected-warning{{Argument to 'delete[]' is offset by 4 bytes from the start of memory allocated by 'new[]'}} |
| 192 | } |
Anton Yartsev | 13df036 | 2013-03-25 01:35:45 +0000 | [diff] [blame] | 193 | |
Anna Zaks | 333481b | 2013-03-28 23:15:29 +0000 | [diff] [blame] | 194 | //-------------------------------- |
| 195 | // Test escape of newed const pointer. Note, a const pointer can be deleted. |
| 196 | //-------------------------------- |
| 197 | struct StWithConstPtr { |
| 198 | const int *memp; |
| 199 | }; |
| 200 | void escape(const int &x); |
| 201 | void escapeStruct(const StWithConstPtr &x); |
| 202 | void escapePtr(const StWithConstPtr *x); |
| 203 | void escapeVoidPtr(const void *x); |
| 204 | |
| 205 | void testConstEscape() { |
| 206 | int *p = new int(1); |
| 207 | escape(*p); |
| 208 | } // no-warning |
| 209 | |
| 210 | void testConstEscapeStruct() { |
| 211 | StWithConstPtr *St = new StWithConstPtr(); |
| 212 | escapeStruct(*St); |
| 213 | } // no-warning |
| 214 | |
| 215 | void testConstEscapeStructPtr() { |
| 216 | StWithConstPtr *St = new StWithConstPtr(); |
| 217 | escapePtr(St); |
| 218 | } // no-warning |
| 219 | |
| 220 | void testConstEscapeMember() { |
| 221 | StWithConstPtr St; |
| 222 | St.memp = new int(2); |
| 223 | escapeVoidPtr(St.memp); |
| 224 | } // no-warning |
| 225 | |
| 226 | void testConstEscapePlacementNew() { |
| 227 | int *x = (int *)malloc(sizeof(int)); |
| 228 | void *y = new (x) int; |
| 229 | escapeVoidPtr(y); |
| 230 | } // no-warning |
Jordan Rose | fbe4d85 | 2013-05-17 02:16:49 +0000 | [diff] [blame] | 231 | |
Jordan Rose | 867b185e | 2013-08-09 00:55:47 +0000 | [diff] [blame] | 232 | //============== Test Uninitialized delete delete[]======================== |
| 233 | void testUninitDelete() { |
| 234 | int *x; |
| 235 | int * y = new int; |
| 236 | delete y; |
| 237 | delete x; // expected-warning{{Argument to 'delete' is uninitialized}} |
| 238 | } |
| 239 | |
| 240 | void testUninitDeleteArray() { |
| 241 | int *x; |
| 242 | int * y = new int[5]; |
| 243 | delete[] y; |
| 244 | delete[] x; // expected-warning{{Argument to 'delete[]' is uninitialized}} |
| 245 | } |
| 246 | |
| 247 | void testUninitFree() { |
| 248 | int *x; |
Daniel Marjamaki | 3d8d6ed | 2017-03-08 15:22:24 +0000 | [diff] [blame] | 249 | free(x); // expected-warning{{1st function call argument is an uninitialized value}} |
Jordan Rose | 867b185e | 2013-08-09 00:55:47 +0000 | [diff] [blame] | 250 | } |
| 251 | |
| 252 | void testUninitDeleteSink() { |
| 253 | int *x; |
| 254 | delete x; // expected-warning{{Argument to 'delete' is uninitialized}} |
| 255 | (*(volatile int *)0 = 1); // no warn |
| 256 | } |
| 257 | |
| 258 | void testUninitDeleteArraySink() { |
| 259 | int *x; |
| 260 | delete[] x; // expected-warning{{Argument to 'delete[]' is uninitialized}} |
| 261 | (*(volatile int *)0 = 1); // no warn |
| 262 | } |
Jordan Rose | fbe4d85 | 2013-05-17 02:16:49 +0000 | [diff] [blame] | 263 | |
| 264 | namespace reference_count { |
| 265 | class control_block { |
| 266 | unsigned count; |
| 267 | public: |
| 268 | control_block() : count(0) {} |
| 269 | void retain() { ++count; } |
| 270 | int release() { return --count; } |
| 271 | }; |
| 272 | |
| 273 | template <typename T> |
| 274 | class shared_ptr { |
| 275 | T *p; |
| 276 | control_block *control; |
| 277 | |
| 278 | public: |
| 279 | shared_ptr() : p(0), control(0) {} |
| 280 | explicit shared_ptr(T *p) : p(p), control(new control_block) { |
| 281 | control->retain(); |
| 282 | } |
| 283 | shared_ptr(shared_ptr &other) : p(other.p), control(other.control) { |
| 284 | if (control) |
| 285 | control->retain(); |
| 286 | } |
| 287 | ~shared_ptr() { |
| 288 | if (control && control->release() == 0) { |
| 289 | delete p; |
| 290 | delete control; |
| 291 | } |
| 292 | }; |
| 293 | |
| 294 | T &operator *() { |
| 295 | return *p; |
| 296 | }; |
| 297 | |
| 298 | void swap(shared_ptr &other) { |
| 299 | T *tmp = p; |
| 300 | p = other.p; |
| 301 | other.p = tmp; |
| 302 | |
| 303 | control_block *ctrlTmp = control; |
| 304 | control = other.control; |
| 305 | other.control = ctrlTmp; |
| 306 | } |
| 307 | }; |
| 308 | |
| 309 | void testSingle() { |
| 310 | shared_ptr<int> a(new int); |
| 311 | *a = 1; |
| 312 | } |
| 313 | |
| 314 | void testDouble() { |
| 315 | shared_ptr<int> a(new int); |
| 316 | shared_ptr<int> b = a; |
| 317 | *a = 1; |
| 318 | } |
| 319 | |
| 320 | void testInvalidated() { |
| 321 | shared_ptr<int> a(new int); |
| 322 | shared_ptr<int> b = a; |
| 323 | *a = 1; |
| 324 | |
| 325 | extern void use(shared_ptr<int> &); |
| 326 | use(b); |
| 327 | } |
| 328 | |
| 329 | void testNestedScope() { |
| 330 | shared_ptr<int> a(new int); |
| 331 | { |
| 332 | shared_ptr<int> b = a; |
| 333 | } |
| 334 | *a = 1; |
| 335 | } |
| 336 | |
| 337 | void testSwap() { |
| 338 | shared_ptr<int> a(new int); |
| 339 | shared_ptr<int> b; |
| 340 | shared_ptr<int> c = a; |
| 341 | shared_ptr<int>(c).swap(b); |
| 342 | } |
| 343 | |
| 344 | void testUseAfterFree() { |
| 345 | int *p = new int; |
| 346 | { |
| 347 | shared_ptr<int> a(p); |
| 348 | shared_ptr<int> b = a; |
| 349 | } |
| 350 | |
| 351 | // FIXME: We should get a warning here, but we don't because we've |
| 352 | // conservatively modeled ~shared_ptr. |
| 353 | *p = 1; |
| 354 | } |
| 355 | } |
| 356 | |
Jordan Rose | 1ccc43d | 2013-09-25 16:06:17 +0000 | [diff] [blame] | 357 | // Test double delete |
| 358 | class DerefClass{ |
| 359 | public: |
| 360 | int *x; |
| 361 | DerefClass() {} |
Jordan Rose | 656fdd5 | 2014-01-08 18:46:55 +0000 | [diff] [blame] | 362 | ~DerefClass() {*x = 1;} |
Jordan Rose | 1ccc43d | 2013-09-25 16:06:17 +0000 | [diff] [blame] | 363 | }; |
| 364 | |
| 365 | void testDoubleDeleteClassInstance() { |
| 366 | DerefClass *foo = new DerefClass(); |
| 367 | delete foo; |
Jordan Rose | 656fdd5 | 2014-01-08 18:46:55 +0000 | [diff] [blame] | 368 | delete foo; // expected-warning {{Attempt to delete released memory}} |
Jordan Rose | 1ccc43d | 2013-09-25 16:06:17 +0000 | [diff] [blame] | 369 | } |
| 370 | |
| 371 | class EmptyClass{ |
| 372 | public: |
| 373 | EmptyClass() {} |
| 374 | ~EmptyClass() {} |
| 375 | }; |
| 376 | |
| 377 | void testDoubleDeleteEmptyClass() { |
| 378 | EmptyClass *foo = new EmptyClass(); |
| 379 | delete foo; |
Jordan Rose | 656fdd5 | 2014-01-08 18:46:55 +0000 | [diff] [blame] | 380 | delete foo; // expected-warning {{Attempt to delete released memory}} |
Jordan Rose | 1ccc43d | 2013-09-25 16:06:17 +0000 | [diff] [blame] | 381 | } |
Gabor Horvath | 44583ce6 | 2016-08-08 09:22:59 +0000 | [diff] [blame] | 382 | |
| 383 | struct Base { |
| 384 | virtual ~Base() {} |
| 385 | }; |
| 386 | |
| 387 | struct Derived : Base { |
| 388 | }; |
| 389 | |
| 390 | Base *allocate() { |
| 391 | return new Derived; |
| 392 | } |
| 393 | |
| 394 | void shouldNotReportLeak() { |
| 395 | Derived *p = (Derived *)allocate(); |
| 396 | delete p; |
| 397 | } |