Ted Kremenek | a0125d8 | 2011-02-16 01:57:07 +0000 | [diff] [blame] | 1 | // RUN: %clang_cc1 -verify %s |
| 2 | |
| 3 | int foo() { |
Ted Kremenek | bac7737 | 2011-02-16 22:08:28 +0000 | [diff] [blame] | 4 | int x[2]; // expected-note 4 {{array 'x' declared here}} |
| 5 | int y[2]; // expected-note 2 {{array 'y' declared here}} |
Chandler Carruth | c268434 | 2011-08-05 09:10:50 +0000 | [diff] [blame] | 6 | int z[1]; // expected-note {{array 'z' declared here}} |
Matt Beaumont-Gay | cfbc5b5 | 2011-11-29 19:27:11 +0000 | [diff] [blame] | 7 | int w[1][1]; // expected-note {{array 'w' declared here}} |
| 8 | int v[1][1][1]; // expected-note {{array 'v' declared here}} |
Ted Kremenek | a0125d8 | 2011-02-16 01:57:07 +0000 | [diff] [blame] | 9 | int *p = &y[2]; // no-warning |
| 10 | (void) sizeof(x[2]); // no-warning |
Matt Beaumont-Gay | a5aa96d | 2011-11-24 00:27:38 +0000 | [diff] [blame] | 11 | y[2] = 2; // expected-warning {{array index 2 is past the end of the array (which contains 2 elements)}} |
| 12 | z[1] = 'x'; // expected-warning {{array index 1 is past the end of the array (which contains 1 element)}} |
Matt Beaumont-Gay | cfbc5b5 | 2011-11-29 19:27:11 +0000 | [diff] [blame] | 13 | w[0][2] = 0; // expected-warning {{array index 2 is past the end of the array (which contains 1 element)}} |
| 14 | v[0][0][2] = 0; // expected-warning {{array index 2 is past the end of the array (which contains 1 element)}} |
Matt Beaumont-Gay | a5aa96d | 2011-11-24 00:27:38 +0000 | [diff] [blame] | 15 | return x[2] + // expected-warning {{array index 2 is past the end of the array (which contains 2 elements)}} |
| 16 | y[-1] + // expected-warning {{array index -1 is before the beginning of the array}} |
| 17 | x[sizeof(x)] + // expected-warning {{array index 8 is past the end of the array (which contains 2 elements)}} |
| 18 | x[sizeof(x) / sizeof(x[0])] + // expected-warning {{array index 2 is past the end of the array (which contains 2 elements)}} |
Ted Kremenek | a0125d8 | 2011-02-16 01:57:07 +0000 | [diff] [blame] | 19 | x[sizeof(x) / sizeof(x[0]) - 1] + // no-warning |
Matt Beaumont-Gay | a5aa96d | 2011-11-24 00:27:38 +0000 | [diff] [blame] | 20 | x[sizeof(x[2])]; // expected-warning {{array index 4 is past the end of the array (which contains 2 elements)}} |
Ted Kremenek | a0125d8 | 2011-02-16 01:57:07 +0000 | [diff] [blame] | 21 | } |
| 22 | |
Ted Kremenek | c71a2c0 | 2011-02-16 23:39:09 +0000 | [diff] [blame] | 23 | // This code example tests that -Warray-bounds works with arrays that |
| 24 | // are template parameters. |
| 25 | template <char *sz> class Qux { |
| 26 | bool test() { return sz[0] == 'a'; } |
Chandler Carruth | 35001ca | 2011-02-17 21:10:52 +0000 | [diff] [blame] | 27 | }; |
| 28 | |
| 29 | void f1(int a[1]) { |
| 30 | int val = a[3]; // no warning for function argumnet |
| 31 | } |
| 32 | |
Chris Lattner | 9e6a1ca | 2011-08-02 21:44:23 +0000 | [diff] [blame] | 33 | void f2(const int (&a)[2]) { // expected-note {{declared here}} |
Matt Beaumont-Gay | a5aa96d | 2011-11-24 00:27:38 +0000 | [diff] [blame] | 34 | int val = a[3]; // expected-warning {{array index 3 is past the end of the array (which contains 2 elements)}} |
Chandler Carruth | 35001ca | 2011-02-17 21:10:52 +0000 | [diff] [blame] | 35 | } |
| 36 | |
| 37 | void test() { |
| 38 | struct { |
| 39 | int a[0]; |
| 40 | } s2; |
| 41 | s2.a[3] = 0; // no warning for 0-sized array |
| 42 | |
| 43 | union { |
Kaelyn Uhrain | d6c8865 | 2011-08-05 23:18:04 +0000 | [diff] [blame] | 44 | short a[2]; // expected-note 4 {{declared here}} |
Chandler Carruth | 35001ca | 2011-02-17 21:10:52 +0000 | [diff] [blame] | 45 | char c[4]; |
| 46 | } u; |
Matt Beaumont-Gay | a5aa96d | 2011-11-24 00:27:38 +0000 | [diff] [blame] | 47 | u.a[3] = 1; // expected-warning {{array index 3 is past the end of the array (which contains 2 elements)}} |
Chandler Carruth | 35001ca | 2011-02-17 21:10:52 +0000 | [diff] [blame] | 48 | u.c[3] = 1; // no warning |
Kaelyn Uhrain | d6c8865 | 2011-08-05 23:18:04 +0000 | [diff] [blame] | 49 | short *p = &u.a[2]; // no warning |
Matt Beaumont-Gay | a5aa96d | 2011-11-24 00:27:38 +0000 | [diff] [blame] | 50 | p = &u.a[3]; // expected-warning {{array index 3 is past the end of the array (which contains 2 elements)}} |
| 51 | *(&u.a[2]) = 1; // expected-warning {{array index 2 is past the end of the array (which contains 2 elements)}} |
| 52 | *(&u.a[3]) = 1; // expected-warning {{array index 3 is past the end of the array (which contains 2 elements)}} |
Kaelyn Uhrain | d6c8865 | 2011-08-05 23:18:04 +0000 | [diff] [blame] | 53 | *(&u.c[3]) = 1; // no warning |
Chandler Carruth | 35001ca | 2011-02-17 21:10:52 +0000 | [diff] [blame] | 54 | |
| 55 | const int const_subscript = 3; |
Chris Lattner | 9e6a1ca | 2011-08-02 21:44:23 +0000 | [diff] [blame] | 56 | int array[2]; // expected-note {{declared here}} |
Matt Beaumont-Gay | a5aa96d | 2011-11-24 00:27:38 +0000 | [diff] [blame] | 57 | array[const_subscript] = 0; // expected-warning {{array index 3 is past the end of the array (which contains 2 elements)}} |
Chandler Carruth | 35001ca | 2011-02-17 21:10:52 +0000 | [diff] [blame] | 58 | |
| 59 | int *ptr; |
| 60 | ptr[3] = 0; // no warning for pointer references |
| 61 | int array2[] = { 0, 1, 2 }; // expected-note 2 {{declared here}} |
| 62 | |
Matt Beaumont-Gay | a5aa96d | 2011-11-24 00:27:38 +0000 | [diff] [blame] | 63 | array2[3] = 0; // expected-warning {{array index 3 is past the end of the array (which contains 3 elements)}} |
| 64 | array2[2+2] = 0; // expected-warning {{array index 4 is past the end of the array (which contains 3 elements)}} |
Chandler Carruth | 35001ca | 2011-02-17 21:10:52 +0000 | [diff] [blame] | 65 | |
| 66 | const char *str1 = "foo"; |
| 67 | char c1 = str1[5]; // no warning for pointers |
| 68 | |
| 69 | const char str2[] = "foo"; // expected-note {{declared here}} |
Matt Beaumont-Gay | a5aa96d | 2011-11-24 00:27:38 +0000 | [diff] [blame] | 70 | char c2 = str2[5]; // expected-warning {{array index 5 is past the end of the array (which contains 4 elements)}} |
Chandler Carruth | 35001ca | 2011-02-17 21:10:52 +0000 | [diff] [blame] | 71 | |
Chris Lattner | 9e6a1ca | 2011-08-02 21:44:23 +0000 | [diff] [blame] | 72 | int (*array_ptr)[2]; |
Matt Beaumont-Gay | a5aa96d | 2011-11-24 00:27:38 +0000 | [diff] [blame] | 73 | (*array_ptr)[3] = 1; // expected-warning {{array index 3 is past the end of the array (which contains 2 elements)}} |
Chandler Carruth | 35001ca | 2011-02-17 21:10:52 +0000 | [diff] [blame] | 74 | } |
| 75 | |
| 76 | template <int I> struct S { |
Ted Kremenek | 351ba91 | 2011-02-23 01:52:04 +0000 | [diff] [blame] | 77 | char arr[I]; // expected-note 2 {{declared here}} |
Chandler Carruth | 35001ca | 2011-02-17 21:10:52 +0000 | [diff] [blame] | 78 | }; |
| 79 | template <int I> void f() { |
| 80 | S<3> s; |
Matt Beaumont-Gay | a5aa96d | 2011-11-24 00:27:38 +0000 | [diff] [blame] | 81 | s.arr[4] = 0; // expected-warning {{array index 4 is past the end of the array (which contains 3 elements)}} |
| 82 | s.arr[I] = 0; // expected-warning {{array index 5 is past the end of the array (which contains 3 elements)}} |
Chandler Carruth | 35001ca | 2011-02-17 21:10:52 +0000 | [diff] [blame] | 83 | } |
| 84 | |
| 85 | void test_templates() { |
| 86 | f<5>(); // expected-note {{in instantiation}} |
| 87 | } |
Ted Kremenek | a85f528 | 2011-02-17 21:40:51 +0000 | [diff] [blame] | 88 | |
| 89 | #define SIZE 10 |
| 90 | #define ARR_IN_MACRO(flag, arr, idx) flag ? arr[idx] : 1 |
| 91 | |
| 92 | int test_no_warn_macro_unreachable() { |
Ted Kremenek | 351ba91 | 2011-02-23 01:52:04 +0000 | [diff] [blame] | 93 | int arr[SIZE]; // expected-note {{array 'arr' declared here}} |
| 94 | return ARR_IN_MACRO(0, arr, SIZE) + // no-warning |
Matt Beaumont-Gay | a5aa96d | 2011-11-24 00:27:38 +0000 | [diff] [blame] | 95 | ARR_IN_MACRO(1, arr, SIZE); // expected-warning{{array index 10 is past the end of the array (which contains 10 elements)}} |
Ted Kremenek | a85f528 | 2011-02-17 21:40:51 +0000 | [diff] [blame] | 96 | } |
| 97 | |
Ted Kremenek | 25b3b84 | 2011-02-18 02:27:00 +0000 | [diff] [blame] | 98 | // This exhibited an assertion failure for a 32-bit build of Clang. |
| 99 | int test_pr9240() { |
| 100 | short array[100]; // expected-note {{array 'array' declared here}} |
Matt Beaumont-Gay | a5aa96d | 2011-11-24 00:27:38 +0000 | [diff] [blame] | 101 | return array[(unsigned long long) 100]; // expected-warning {{array index 100 is past the end of the array (which contains 100 elements)}} |
Ted Kremenek | 25b3b84 | 2011-02-18 02:27:00 +0000 | [diff] [blame] | 102 | } |
| 103 | |
Ted Kremenek | 3bcc2be | 2011-02-23 01:52:07 +0000 | [diff] [blame] | 104 | // PR 9284 - a template parameter can cause an array bounds access to be |
| 105 | // infeasible. |
Ted Kremenek | 351ba91 | 2011-02-23 01:52:04 +0000 | [diff] [blame] | 106 | template <bool extendArray> |
Ted Kremenek | 3bcc2be | 2011-02-23 01:52:07 +0000 | [diff] [blame] | 107 | void pr9284() { |
Ted Kremenek | 351ba91 | 2011-02-23 01:52:04 +0000 | [diff] [blame] | 108 | int arr[3 + (extendArray ? 1 : 0)]; |
| 109 | |
| 110 | if (extendArray) |
Ted Kremenek | 3bcc2be | 2011-02-23 01:52:07 +0000 | [diff] [blame] | 111 | arr[3] = 42; // no-warning |
Ted Kremenek | 351ba91 | 2011-02-23 01:52:04 +0000 | [diff] [blame] | 112 | } |
| 113 | |
Ted Kremenek | 3bcc2be | 2011-02-23 01:52:07 +0000 | [diff] [blame] | 114 | template <bool extendArray> |
| 115 | void pr9284b() { |
| 116 | int arr[3 + (extendArray ? 1 : 0)]; // expected-note {{array 'arr' declared here}} |
| 117 | |
| 118 | if (!extendArray) |
Matt Beaumont-Gay | a5aa96d | 2011-11-24 00:27:38 +0000 | [diff] [blame] | 119 | arr[3] = 42; // expected-warning{{array index 3 is past the end of the array (which contains 3 elements)}} |
Ted Kremenek | 3bcc2be | 2011-02-23 01:52:07 +0000 | [diff] [blame] | 120 | } |
| 121 | |
| 122 | void test_pr9284() { |
| 123 | pr9284<true>(); |
| 124 | pr9284<false>(); |
| 125 | pr9284b<true>(); |
| 126 | pr9284b<false>(); // expected-note{{in instantiation of function template specialization 'pr9284b<false>' requested here}} |
Ted Kremenek | 351ba91 | 2011-02-23 01:52:04 +0000 | [diff] [blame] | 127 | } |
| 128 | |
Ted Kremenek | 9e060ca | 2011-02-23 23:06:04 +0000 | [diff] [blame] | 129 | int test_pr9296() { |
| 130 | int array[2]; |
| 131 | return array[true]; // no-warning |
| 132 | } |
| 133 | |
Ted Kremenek | 3aea4da | 2011-03-01 18:41:00 +0000 | [diff] [blame] | 134 | int test_sizeof_as_condition(int flag) { |
| 135 | int arr[2] = { 0, 0 }; // expected-note {{array 'arr' declared here}} |
| 136 | if (flag) |
| 137 | return sizeof(char) != sizeof(char) ? arr[2] : arr[1]; |
Matt Beaumont-Gay | a5aa96d | 2011-11-24 00:27:38 +0000 | [diff] [blame] | 138 | return sizeof(char) == sizeof(char) ? arr[2] : arr[1]; // expected-warning {{array index 2 is past the end of the array (which contains 2 elements)}} |
Ted Kremenek | 3aea4da | 2011-03-01 18:41:00 +0000 | [diff] [blame] | 139 | } |
| 140 | |
Ted Kremenek | e71f3d5 | 2011-03-01 23:12:55 +0000 | [diff] [blame] | 141 | void test_switch() { |
| 142 | switch (4) { |
| 143 | case 1: { |
| 144 | int arr[2]; |
| 145 | arr[2] = 1; // no-warning |
| 146 | break; |
| 147 | } |
| 148 | case 4: { |
| 149 | int arr[2]; // expected-note {{array 'arr' declared here}} |
Matt Beaumont-Gay | a5aa96d | 2011-11-24 00:27:38 +0000 | [diff] [blame] | 150 | arr[2] = 1; // expected-warning {{array index 2 is past the end of the array (which contains 2 elements)}} |
Ted Kremenek | e71f3d5 | 2011-03-01 23:12:55 +0000 | [diff] [blame] | 151 | break; |
| 152 | } |
| 153 | default: { |
| 154 | int arr[2]; |
| 155 | arr[2] = 1; // no-warning |
| 156 | break; |
| 157 | } |
| 158 | } |
| 159 | } |
Ted Kremenek | 3aea4da | 2011-03-01 18:41:00 +0000 | [diff] [blame] | 160 | |
Ted Kremenek | 0498247 | 2011-03-04 01:03:41 +0000 | [diff] [blame] | 161 | // Test nested switch statements. |
| 162 | enum enumA { enumA_A, enumA_B, enumA_C, enumA_D, enumA_E }; |
| 163 | enum enumB { enumB_X, enumB_Y, enumB_Z }; |
| 164 | static enum enumB myVal = enumB_X; |
Kaelyn Uhrain | d6c8865 | 2011-08-05 23:18:04 +0000 | [diff] [blame] | 165 | void test_nested_switch() { |
Ted Kremenek | 0498247 | 2011-03-04 01:03:41 +0000 | [diff] [blame] | 166 | switch (enumA_E) { // expected-warning {{no case matching constant}} |
| 167 | switch (myVal) { // expected-warning {{enumeration values 'enumB_X' and 'enumB_Z' not handled in switch}} |
| 168 | case enumB_Y: ; |
| 169 | } |
| 170 | } |
| 171 | } |
| 172 | |
Ted Kremenek | 432c478 | 2011-03-16 04:32:01 +0000 | [diff] [blame] | 173 | // Test that if all the values of an enum covered, that the 'default' branch |
| 174 | // is unreachable. |
| 175 | enum Values { A, B, C, D }; |
| 176 | void test_all_enums_covered(enum Values v) { |
| 177 | int x[2]; |
| 178 | switch (v) { |
| 179 | case A: return; |
| 180 | case B: return; |
| 181 | case C: return; |
| 182 | case D: return; |
| 183 | } |
| 184 | x[2] = 0; // no-warning |
| 185 | } |
Chris Lattner | 9e6a1ca | 2011-08-02 21:44:23 +0000 | [diff] [blame] | 186 | |
| 187 | namespace tailpad { |
| 188 | struct foo { |
Chandler Carruth | c268434 | 2011-08-05 09:10:50 +0000 | [diff] [blame] | 189 | char c1[1]; // expected-note {{declared here}} |
Chris Lattner | 9e6a1ca | 2011-08-02 21:44:23 +0000 | [diff] [blame] | 190 | int x; |
Chandler Carruth | c268434 | 2011-08-05 09:10:50 +0000 | [diff] [blame] | 191 | char c2[1]; |
Chris Lattner | 9e6a1ca | 2011-08-02 21:44:23 +0000 | [diff] [blame] | 192 | }; |
Matt Beaumont-Gay | 381711c | 2011-11-29 22:43:53 +0000 | [diff] [blame^] | 193 | |
| 194 | class baz { |
| 195 | public: |
| 196 | char c1[1]; // expected-note {{declared here}} |
| 197 | int x; |
| 198 | char c2[1]; |
| 199 | }; |
| 200 | |
| 201 | char bar(struct foo *F, baz *B) { |
| 202 | return F->c1[3] + // expected-warning {{array index 3 is past the end of the array (which contains 1 element)}} |
| 203 | F->c2[3] + // no warning, foo could have tail padding allocated. |
| 204 | B->c1[3] + // expected-warning {{array index 3 is past the end of the array (which contains 1 element)}} |
| 205 | B->c2[3]; // no warning, baz could have tail padding allocated. |
Chandler Carruth | c268434 | 2011-08-05 09:10:50 +0000 | [diff] [blame] | 206 | } |
| 207 | } |
| 208 | |
| 209 | namespace metaprogramming { |
| 210 | #define ONE 1 |
| 211 | struct foo { char c[ONE]; }; // expected-note {{declared here}} |
| 212 | template <int N> struct bar { char c[N]; }; // expected-note {{declared here}} |
| 213 | |
| 214 | char test(foo *F, bar<1> *B) { |
Matt Beaumont-Gay | a5aa96d | 2011-11-24 00:27:38 +0000 | [diff] [blame] | 215 | return F->c[3] + // expected-warning {{array index 3 is past the end of the array (which contains 1 element)}} |
| 216 | B->c[3]; // expected-warning {{array index 3 is past the end of the array (which contains 1 element)}} |
Chris Lattner | 9e6a1ca | 2011-08-02 21:44:23 +0000 | [diff] [blame] | 217 | } |
| 218 | } |
Kaelyn Uhrain | d6c8865 | 2011-08-05 23:18:04 +0000 | [diff] [blame] | 219 | |
| 220 | void bar(int x) {} |
| 221 | int test_more() { |
| 222 | int foo[5]; // expected-note 5 {{array 'foo' declared here}} |
Matt Beaumont-Gay | a5aa96d | 2011-11-24 00:27:38 +0000 | [diff] [blame] | 223 | bar(foo[5]); // expected-warning {{array index 5 is past the end of the array (which contains 5 elements)}} |
| 224 | ++foo[5]; // expected-warning {{array index 5 is past the end of the array (which contains 5 elements)}} |
| 225 | if (foo[6]) // expected-warning {{array index 6 is past the end of the array (which contains 5 elements)}} |
| 226 | return --foo[6]; // expected-warning {{array index 6 is past the end of the array (which contains 5 elements)}} |
Kaelyn Uhrain | d6c8865 | 2011-08-05 23:18:04 +0000 | [diff] [blame] | 227 | else |
Matt Beaumont-Gay | a5aa96d | 2011-11-24 00:27:38 +0000 | [diff] [blame] | 228 | return foo[5]; // expected-warning {{array index 5 is past the end of the array (which contains 5 elements)}} |
Kaelyn Uhrain | d6c8865 | 2011-08-05 23:18:04 +0000 | [diff] [blame] | 229 | } |
Nico Weber | de5998f | 2011-09-17 22:59:41 +0000 | [diff] [blame] | 230 | |
| 231 | void test_pr10771() { |
| 232 | double foo[4096]; // expected-note {{array 'foo' declared here}} |
| 233 | |
| 234 | ((char*)foo)[sizeof(foo) - 1] = '\0'; // no-warning |
| 235 | *(((char*)foo) + sizeof(foo) - 1) = '\0'; // no-warning |
| 236 | |
Matt Beaumont-Gay | a5aa96d | 2011-11-24 00:27:38 +0000 | [diff] [blame] | 237 | ((char*)foo)[sizeof(foo)] = '\0'; // expected-warning {{array index 32768 is past the end of the array (which contains 32768 elements)}} |
Nico Weber | de5998f | 2011-09-17 22:59:41 +0000 | [diff] [blame] | 238 | |
| 239 | // TODO: This should probably warn, too. |
| 240 | *(((char*)foo) + sizeof(foo)) = '\0'; // no-warning |
| 241 | } |
Ted Kremenek | 615eb7c | 2011-09-26 23:36:13 +0000 | [diff] [blame] | 242 | |
| 243 | int test_pr11007_aux(const char * restrict, ...); |
| 244 | |
| 245 | // Test checking with varargs. |
| 246 | void test_pr11007() { |
| 247 | double a[5]; // expected-note {{array 'a' declared here}} |
Matt Beaumont-Gay | a5aa96d | 2011-11-24 00:27:38 +0000 | [diff] [blame] | 248 | test_pr11007_aux("foo", a[1000]); // expected-warning {{array index 1000 is past the end of the array}} |
Ted Kremenek | 615eb7c | 2011-09-26 23:36:13 +0000 | [diff] [blame] | 249 | } |