Ted Kremenek | 8baf86d | 2009-07-23 21:34:35 +0000 | [diff] [blame^] | 1 | // RUN: clang-cc -analyze -warn-security-syntactic %s -verify |
| 2 | |
| 3 | // <rdar://problem/6336718> rule request: floating point used as loop |
| 4 | // condition (FLP30-C, FLP-30-CPP) |
| 5 | // |
| 6 | // For reference: https://www.securecoding.cert.org/confluence/display/seccode/FLP30-C.+Do+not+use+floating+point+variables+as+loop+counters |
| 7 | // |
| 8 | void test_float_condition() { |
| 9 | for (float x = 0.1f; x <= 1.0f; x += 0.1f) {} // expected-warning{{Variable 'x' with floating point type 'float'}} |
| 10 | for (float x = 100000001.0f; x <= 100000010.0f; x += 1.0f) {} // expected-warning{{Variable 'x' with floating point type 'float'}} |
| 11 | for (float x = 100000001.0f; x <= 100000010.0f; x++ ) {} // expected-warning{{Variable 'x' with floating point type 'float'}} |
| 12 | for (double x = 100000001.0; x <= 100000010.0; x++ ) {} // expected-warning{{Variable 'x' with floating point type 'double'}} |
| 13 | for (double x = 100000001.0; ((x)) <= 100000010.0; ((x))++ ) {} // expected-warning{{Variable 'x' with floating point type 'double'}} |
| 14 | |
| 15 | for (double x = 100000001.0; 100000010.0 >= x; x = x + 1.0 ) {} // expected-warning{{Variable 'x' with floating point type 'double'}} |
| 16 | |
| 17 | int i = 0; |
| 18 | for (double x = 100000001.0; ((x)) <= 100000010.0; ((x))++, ++i ) {} // expected-warning{{Variable 'x' with floating point type 'double'}} |
| 19 | |
| 20 | typedef float FooType; |
| 21 | for (FooType x = 100000001.0f; x <= 100000010.0f; x++ ) {} // expected-warning{{Variable 'x' with floating point type 'FooType'}} |
| 22 | } |
| 23 | |