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