blob: 49af4f32262959e855c015bb5105ce4aa08b3661 [file] [log] [blame]
Ted Kremenek0a65f942011-03-17 03:06:07 +00001// RUN: %clang_cc1 -fsyntax-only -Wuninitialized -Wconditional-uninitialized -fsyntax-only -fblocks %s -verify
Ted Kremenek610068c2011-01-15 02:58:47 +00002
Ted Kremenek540dda62011-08-23 20:30:50 +00003typedef __typeof(sizeof(int)) size_t;
4void *malloc(size_t);
5
Ted Kremenek610068c2011-01-15 02:58:47 +00006int test1() {
David Blaikie4f4f3492011-09-10 05:35:08 +00007 int x; // expected-note{{initialize the variable 'x' to silence this warning}}
Chandler Carruthf04eb2d2011-04-08 06:33:38 +00008 return x; // expected-warning{{variable 'x' is uninitialized when used here}}
Ted Kremenek610068c2011-01-15 02:58:47 +00009}
10
11int test2() {
12 int x = 0;
13 return x; // no-warning
14}
15
16int test3() {
17 int x;
18 x = 0;
19 return x; // no-warning
20}
21
22int test4() {
David Blaikie4f4f3492011-09-10 05:35:08 +000023 int x; // expected-note{{initialize the variable 'x' to silence this warning}}
Chandler Carruthf04eb2d2011-04-08 06:33:38 +000024 ++x; // expected-warning{{variable 'x' is uninitialized when used here}}
Ted Kremenek610068c2011-01-15 02:58:47 +000025 return x;
26}
27
28int test5() {
David Blaikie4f4f3492011-09-10 05:35:08 +000029 int x, y; // expected-note{{initialize the variable 'y' to silence this warning}}
Chandler Carruthf04eb2d2011-04-08 06:33:38 +000030 x = y; // expected-warning{{variable 'y' is uninitialized when used here}}
Ted Kremenek610068c2011-01-15 02:58:47 +000031 return x;
32}
33
34int test6() {
David Blaikie4f4f3492011-09-10 05:35:08 +000035 int x; // expected-note{{initialize the variable 'x' to silence this warning}}
Chandler Carruthf04eb2d2011-04-08 06:33:38 +000036 x += 2; // expected-warning{{variable 'x' is uninitialized when used here}}
Ted Kremenek610068c2011-01-15 02:58:47 +000037 return x;
38}
39
40int test7(int y) {
David Blaikie4f4f3492011-09-10 05:35:08 +000041 int x; // expected-note{{initialize the variable 'x' to silence this warning}}
Ted Kremenek610068c2011-01-15 02:58:47 +000042 if (y)
43 x = 1;
Chandler Carruth584b9d62011-04-08 06:47:15 +000044 return x; // expected-warning{{variable 'x' may be uninitialized when used here}}
Ted Kremenek610068c2011-01-15 02:58:47 +000045}
46
47int test8(int y) {
48 int x;
49 if (y)
50 x = 1;
51 else
52 x = 0;
Ted Kremenek609e3172011-02-02 23:35:53 +000053 return x;
Ted Kremenek610068c2011-01-15 02:58:47 +000054}
55
56int test9(int n) {
David Blaikie4f4f3492011-09-10 05:35:08 +000057 int x; // expected-note{{initialize the variable 'x' to silence this warning}}
Ted Kremenek610068c2011-01-15 02:58:47 +000058 for (unsigned i = 0 ; i < n; ++i) {
59 if (i == n - 1)
60 break;
Ted Kremenek94b1b4d2011-01-21 19:41:41 +000061 x = 1;
Ted Kremenek610068c2011-01-15 02:58:47 +000062 }
Chandler Carruth584b9d62011-04-08 06:47:15 +000063 return x; // expected-warning{{variable 'x' may be uninitialized when used here}}
Ted Kremenek610068c2011-01-15 02:58:47 +000064}
65
66int test10(unsigned n) {
David Blaikie4f4f3492011-09-10 05:35:08 +000067 int x; // expected-note{{initialize the variable 'x' to silence this warning}}
Ted Kremenek610068c2011-01-15 02:58:47 +000068 for (unsigned i = 0 ; i < n; ++i) {
69 x = 1;
70 }
Chandler Carruth584b9d62011-04-08 06:47:15 +000071 return x; // expected-warning{{variable 'x' may be uninitialized when used here}}
Ted Kremenek610068c2011-01-15 02:58:47 +000072}
73
74int test11(unsigned n) {
David Blaikie4f4f3492011-09-10 05:35:08 +000075 int x; // expected-note{{initialize the variable 'x' to silence this warning}}
Ted Kremenek610068c2011-01-15 02:58:47 +000076 for (unsigned i = 0 ; i <= n; ++i) {
77 x = 1;
78 }
Chandler Carruth584b9d62011-04-08 06:47:15 +000079 return x; // expected-warning{{variable 'x' may be uninitialized when used here}}
Ted Kremenek610068c2011-01-15 02:58:47 +000080}
81
82void test12(unsigned n) {
David Blaikie4f4f3492011-09-10 05:35:08 +000083 for (unsigned i ; n ; ++i) ; // expected-warning{{variable 'i' is uninitialized when used here}} expected-note{{initialize the variable 'i' to silence this warning}}
Ted Kremenek610068c2011-01-15 02:58:47 +000084}
85
86int test13() {
87 static int i;
88 return i; // no-warning
89}
90
Ted Kremenekc104e532011-01-18 04:53:25 +000091// Simply don't crash on this test case.
92void test14() {
93 const char *p = 0;
94 for (;;) {}
95}
Ted Kremenek610068c2011-01-15 02:58:47 +000096
Ted Kremenek9e761722011-10-13 18:50:06 +000097void test15() {
98 int x = x; // no-warning: signals intended lack of initialization.
99}
100
101int test15b() {
102 // Warn here with the self-init, since it does result in a use of
103 // an unintialized variable and this is the root cause.
104 int x = x; // expected-warning {{variable 'x' is uninitialized when used within its own initialization}}
105 return x;
Ted Kremenekc104e532011-01-18 04:53:25 +0000106}
107
108// Don't warn in the following example; shows dataflow confluence.
109char *test16_aux();
110void test16() {
111 char *p = test16_aux();
112 for (unsigned i = 0 ; i < 100 ; i++)
113 p[i] = 'a'; // no-warning
114}
Ted Kremenekc21fed32011-01-18 21:18:58 +0000115
116void test17() {
117 // Don't warn multiple times about the same uninitialized variable
118 // along the same path.
David Blaikie4f4f3492011-09-10 05:35:08 +0000119 int *x; // expected-note{{initialize the variable 'x' to silence this warning}}
Chandler Carruthf04eb2d2011-04-08 06:33:38 +0000120 *x = 1; // expected-warning{{variable 'x' is uninitialized when used here}}
Ted Kremenekc21fed32011-01-18 21:18:58 +0000121 *x = 1; // no-warning
122}
Ted Kremenek13bd4232011-01-20 17:37:17 +0000123
124int test18(int x, int y) {
125 int z;
126 if (x && y && (z = 1)) {
127 return z; // no-warning
128 }
129 return 0;
130}
131
132int test19_aux1();
133int test19_aux2();
134int test19_aux3(int *x);
135int test19() {
136 int z;
137 if (test19_aux1() + test19_aux2() && test19_aux1() && test19_aux3(&z))
138 return z; // no-warning
139 return 0;
140}
141
142int test20() {
David Blaikie4f4f3492011-09-10 05:35:08 +0000143 int z; // expected-note{{initialize the variable 'z' to silence this warning}}
Ted Kremenek13bd4232011-01-20 17:37:17 +0000144 if ((test19_aux1() + test19_aux2() && test19_aux1()) || test19_aux3(&z))
Chandler Carruth584b9d62011-04-08 06:47:15 +0000145 return z; // expected-warning{{variable 'z' may be uninitialized when used here}}
Ted Kremenek13bd4232011-01-20 17:37:17 +0000146 return 0;
147}
148
149int test21(int x, int y) {
David Blaikie4f4f3492011-09-10 05:35:08 +0000150 int z; // expected-note{{initialize the variable 'z' to silence this warning}}
Ted Kremenek13bd4232011-01-20 17:37:17 +0000151 if ((x && y) || test19_aux3(&z) || test19_aux2())
Chandler Carruth584b9d62011-04-08 06:47:15 +0000152 return z; // expected-warning{{variable 'z' may be uninitialized when used here}}
Ted Kremenek13bd4232011-01-20 17:37:17 +0000153 return 0;
154}
155
156int test22() {
157 int z;
158 while (test19_aux1() + test19_aux2() && test19_aux1() && test19_aux3(&z))
159 return z; // no-warning
160 return 0;
161}
162
163int test23() {
164 int z;
165 for ( ; test19_aux1() + test19_aux2() && test19_aux1() && test19_aux3(&z) ; )
166 return z; // no-warning
167 return 0;
168}
169
170// The basic uninitialized value analysis doesn't have enough path-sensitivity
171// to catch initializations relying on control-dependencies spanning multiple
172// conditionals. This possibly can be handled by making the CFG itself
173// represent such control-dependencies, but it is a niche case.
174int test24(int flag) {
David Blaikie4f4f3492011-09-10 05:35:08 +0000175 unsigned val; // expected-note{{initialize the variable 'val' to silence this warning}}
Ted Kremenek13bd4232011-01-20 17:37:17 +0000176 if (flag)
177 val = 1;
178 if (!flag)
179 val = 1;
Chandler Carruth584b9d62011-04-08 06:47:15 +0000180 return val; // expected-warning{{variable 'val' may be uninitialized when used here}}
Ted Kremenek13bd4232011-01-20 17:37:17 +0000181}
182
Ted Kremenekdcfb3602011-01-21 22:49:49 +0000183float test25() {
David Blaikie4f4f3492011-09-10 05:35:08 +0000184 float x; // expected-note{{initialize the variable 'x' to silence this warning}}
Chandler Carruthf04eb2d2011-04-08 06:33:38 +0000185 return x; // expected-warning{{variable 'x' is uninitialized when used here}}
Ted Kremenekdcfb3602011-01-21 22:49:49 +0000186}
187
188typedef int MyInt;
189MyInt test26() {
David Blaikie4f4f3492011-09-10 05:35:08 +0000190 MyInt x; // expected-note{{initialize the variable 'x' to silence this warning}}
Chandler Carruthf04eb2d2011-04-08 06:33:38 +0000191 return x; // expected-warning{{variable 'x' is uninitialized when used here}}
Ted Kremenekdcfb3602011-01-21 22:49:49 +0000192}
Ted Kremenek96608032011-01-23 17:53:04 +0000193
194// Test handling of sizeof().
195int test27() {
196 struct test_27 { int x; } *y;
197 return sizeof(y->x); // no-warning
198}
199
200int test28() {
David Blaikie4f4f3492011-09-10 05:35:08 +0000201 int len; // expected-note{{initialize the variable 'len' to silence this warning}}
Chandler Carruthf04eb2d2011-04-08 06:33:38 +0000202 return sizeof(int[len]); // expected-warning{{variable 'len' is uninitialized when used here}}
Ted Kremenek96608032011-01-23 17:53:04 +0000203}
204
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000205void test29() {
David Blaikie4f4f3492011-09-10 05:35:08 +0000206 int x; // expected-note{{initialize the variable 'x' to silence this warning}}
Chandler Carruthf04eb2d2011-04-08 06:33:38 +0000207 (void) ^{ (void) x; }; // expected-warning{{variable 'x' is uninitialized when captured by block}}
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000208}
209
210void test30() {
211 static int x; // no-warning
212 (void) ^{ (void) x; };
213}
214
215void test31() {
216 __block int x; // no-warning
217 (void) ^{ (void) x; };
218}
219
220int test32_x;
221void test32() {
222 (void) ^{ (void) test32_x; }; // no-warning
223}
224
Ted Kremenekdd0f7942011-01-26 04:49:43 +0000225void test_33() {
226 int x; // no-warning
227 (void) x;
228}
229
230int test_34() {
David Blaikie4f4f3492011-09-10 05:35:08 +0000231 int x; // expected-note{{initialize the variable 'x' to silence this warning}}
Ted Kremenekdd0f7942011-01-26 04:49:43 +0000232 (void) x;
Chandler Carruthf04eb2d2011-04-08 06:33:38 +0000233 return x; // expected-warning{{variable 'x' is uninitialized when used here}}
Ted Kremenekdd0f7942011-01-26 04:49:43 +0000234}
235
Ted Kremenek40900ee2011-01-27 02:29:34 +0000236// Test that this case doesn't crash.
237void test35(int x) {
238 __block int y = 0;
239 ^{ y = (x == 0); }();
240}
241
Ted Kremenek96554fd2011-01-27 18:51:39 +0000242// Test handling of indirect goto.
243void test36()
244{
David Blaikie4f4f3492011-09-10 05:35:08 +0000245 void **pc; // expected-note{{initialize the variable 'pc' to silence this warning}}
Ted Kremenek96554fd2011-01-27 18:51:39 +0000246 void *dummy[] = { &&L1, &&L2 };
247 L1:
Chandler Carruthd837c0d2011-07-22 05:27:52 +0000248 goto *pc; // expected-warning{{variable 'pc' is uninitialized when used here}}
Ted Kremenek96554fd2011-01-27 18:51:39 +0000249 L2:
250 goto *pc;
251}
252
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000253// Test && nested in ||.
254int test37_a();
255int test37_b();
256int test37()
257{
258 int identifier;
259 if ((test37_a() && (identifier = 1)) ||
260 (test37_b() && (identifier = 2))) {
261 return identifier; // no-warning
262 }
263 return 0;
264}
265
266// Test merging of path-specific dataflow values (without asserting).
267int test38(int r, int x, int y)
268{
269 int z;
270 return ((r < 0) || ((r == 0) && (x < y)));
271}
272
Ted Kremenekf3f53792011-03-15 03:17:01 +0000273int test39(int x) {
David Blaikie4f4f3492011-09-10 05:35:08 +0000274 int y; // expected-note{{initialize the variable 'y' to silence this warning}}
Chandler Carruthf04eb2d2011-04-08 06:33:38 +0000275 int z = x + y; // expected-warning {{variable 'y' is uninitialized when used here}}
Ted Kremenekf3f53792011-03-15 03:17:01 +0000276 return z;
277}
278
279
280int test40(int x) {
David Blaikie4f4f3492011-09-10 05:35:08 +0000281 int y; // expected-note{{initialize the variable 'y' to silence this warning}}
Chandler Carruthf04eb2d2011-04-08 06:33:38 +0000282 return x ? 1 : y; // expected-warning {{variable 'y' is uninitialized when used here}}
Ted Kremenekf3f53792011-03-15 03:17:01 +0000283}
284
285int test41(int x) {
David Blaikie4f4f3492011-09-10 05:35:08 +0000286 int y; // expected-note{{initialize the variable 'y' to silence this warning}}
Ted Kremenekf3f53792011-03-15 03:17:01 +0000287 if (x) y = 1; // no-warning
Chandler Carruth584b9d62011-04-08 06:47:15 +0000288 return y; // expected-warning {{variable 'y' may be uninitialized when used here}}
Ted Kremenekf3f53792011-03-15 03:17:01 +0000289}
290
291void test42() {
292 int a;
293 a = 30; // no-warning
294}
295
296void test43_aux(int x);
297void test43(int i) {
David Blaikie4f4f3492011-09-10 05:35:08 +0000298 int x; // expected-note{{initialize the variable 'x' to silence this warning}}
Ted Kremenekf3f53792011-03-15 03:17:01 +0000299 for (i = 0 ; i < 10; i++)
Chandler Carruthd837c0d2011-07-22 05:27:52 +0000300 test43_aux(x++); // expected-warning {{variable 'x' is uninitialized when used here}}
Ted Kremenekf3f53792011-03-15 03:17:01 +0000301}
302
303void test44(int i) {
304 int x = i;
David Blaikie4f4f3492011-09-10 05:35:08 +0000305 int y; // expected-note{{initialize the variable 'y' to silence this warning}}
Ted Kremenekf3f53792011-03-15 03:17:01 +0000306 for (i = 0; i < 10; i++ ) {
307 test43_aux(x++); // no-warning
Chandler Carruthd837c0d2011-07-22 05:27:52 +0000308 x += y; // expected-warning {{variable 'y' is uninitialized when used here}}
Ted Kremenekf3f53792011-03-15 03:17:01 +0000309 }
310}
311
312int test45(int j) {
313 int x = 1, y = x + 1;
314 if (y) // no-warning
315 return x;
316 return y;
317}
318
319void test46()
320{
David Blaikie4f4f3492011-09-10 05:35:08 +0000321 int i; // expected-note{{initialize the variable 'i' to silence this warning}}
Chandler Carruthf04eb2d2011-04-08 06:33:38 +0000322 int j = i ? : 1; // expected-warning {{variable 'i' is uninitialized when used here}}
Ted Kremenekf3f53792011-03-15 03:17:01 +0000323}
324
325void *test47(int *i)
326{
327 return i ? : 0; // no-warning
328}
329
330void *test49(int *i)
331{
332 int a;
333 return &a ? : i; // no-warning
334}
335
336void test50()
337{
338 char c[1 ? : 2]; // no-warning
339}
340
Ted Kremenekbc8b44c2011-03-31 22:32:41 +0000341int test51(void)
342{
343 __block int a;
344 ^(void) {
345 a = 42;
346 }();
347 return a; // no-warning
348}
349
Ted Kremeneke6c28032011-05-10 22:10:35 +0000350// FIXME: This is a false positive, but it tests logical operations in switch statements.
351int test52(int a, int b) {
David Blaikie4f4f3492011-09-10 05:35:08 +0000352 int x; // expected-note {{initialize the variable 'x' to silence this warning}}
Ted Kremeneke6c28032011-05-10 22:10:35 +0000353 switch (a || b) { // expected-warning {{switch condition has boolean value}}
354 case 0:
355 x = 1;
356 break;
357 case 1:
358 x = 2;
359 break;
360 }
361 return x; // expected-warning {{variable 'x' may be uninitialized when used here}}
362}
Chandler Carruth84350692011-07-16 22:27:02 +0000363
Ted Kremenekd626ec42011-07-19 20:33:49 +0000364void test53() {
David Blaikie4f4f3492011-09-10 05:35:08 +0000365 int x; // expected-note {{initialize the variable 'x' to silence this warning}}
Ted Kremenek62d126e2011-07-19 21:41:51 +0000366 int y = (x); // expected-warning {{variable 'x' is uninitialized when used here}}
Ted Kremenekd626ec42011-07-19 20:33:49 +0000367}
368
Chandler Carruth84350692011-07-16 22:27:02 +0000369// This CFG caused the uninitialized values warning to inf-loop.
370extern int PR10379_g();
371void PR10379_f(int *len) {
David Blaikie4f4f3492011-09-10 05:35:08 +0000372 int new_len; // expected-note{{initialize the variable 'new_len' to silence this warning}}
Chandler Carruth84350692011-07-16 22:27:02 +0000373 for (int i = 0; i < 42 && PR10379_g() == 0; i++) {
374 if (PR10379_g() == 1)
375 continue;
376 if (PR10379_g() == 2)
377 PR10379_f(&new_len);
378 else if (PR10379_g() == 3)
379 PR10379_f(&new_len);
380 *len += new_len; // expected-warning {{variable 'new_len' may be uninitialized when used here}}
381 }
382}
Ted Kremenek540dda62011-08-23 20:30:50 +0000383
384// Test that sizeof(VLA) doesn't trigger a warning.
385void test_vla_sizeof(int x) {
386 double (*memory)[2][x] = malloc(sizeof(*memory)); // no-warning
387}
388
Ted Kremenek6f275422011-09-02 19:39:26 +0000389// Test absurd case of deadcode + use of blocks. This previously was a false positive
390// due to an analysis bug.
391int test_block_and_dead_code() {
392 __block int x;
393 ^{ x = 1; }();
394 if (0)
395 return x;
396 return x; // no-warning
397}
398
Ted Kremenekc5f740e2011-10-07 00:42:48 +0000399// This previously triggered an infinite loop in the analysis.
400void PR11069(int a, int b) {
401 unsigned long flags;
402 for (;;) {
403 if (a && !b)
404 break;
405 }
406 for (;;) {
407 // This does not trigger a warning because it isn't a real use.
408 (void)(flags); // no-warning
409 }
410}
411
Ted Kremenekaa2176b2011-10-07 00:52:56 +0000412// Test uninitialized value used in loop condition.
413void rdar9432305(float *P) {
414 int i; // expected-note {{initialize the variable 'i' to silence this warning}}
415 for (; i < 10000; ++i) // expected-warning {{variable 'i' is uninitialized when used here}}
416 P[i] = 0.0f;
417}
Ted Kremenekc5f740e2011-10-07 00:42:48 +0000418