diff options
| author | Ted Kremenek <kremenek@apple.com> | 2011-08-23 20:30:50 +0000 |
|---|---|---|
| committer | Ted Kremenek <kremenek@apple.com> | 2011-08-23 20:30:50 +0000 |
| commit | ee9848e20d2659f939fb6e773729c6a86e6ed1da (patch) | |
| tree | 9450556c0201c0061b7da6dc3c44582f5a0b79d9 /clang/test/Analysis/outofbound-notwork.c | |
| parent | 43c322b5b2de0b89d2b282f5d98785b269a6e2b3 (diff) | |
| download | bcm5719-llvm-ee9848e20d2659f939fb6e773729c6a86e6ed1da.tar.gz bcm5719-llvm-ee9848e20d2659f939fb6e773729c6a86e6ed1da.zip | |
Fix regression in -Wuninitialized involving VLAs. It turns out that we were modeling sizeof(VLAs)
incorrectly in the CFG, and also the static analyzer. This patch regresses the analyzer a bit, but
that needs to be followed up with a better solution.
Fixes <rdar://problem/10008112>.
llvm-svn: 138372
Diffstat (limited to 'clang/test/Analysis/outofbound-notwork.c')
| -rw-r--r-- | clang/test/Analysis/outofbound-notwork.c | 32 |
1 files changed, 32 insertions, 0 deletions
diff --git a/clang/test/Analysis/outofbound-notwork.c b/clang/test/Analysis/outofbound-notwork.c new file mode 100644 index 00000000000..45e713b6880 --- /dev/null +++ b/clang/test/Analysis/outofbound-notwork.c @@ -0,0 +1,32 @@ +// RUN: %clang_cc1 -Wno-array-bounds -analyze -analyzer-checker=core,experimental.unix,experimental.security.ArrayBound -analyzer-store=region -verify %s +// XFAIL: * + +// Once we better handle modeling of sizes of VLAs, we can pull this back +// into outofbound.c. + +void sizeof_vla(int a) { + if (a == 5) { + char x[a]; + int y[sizeof(x)]; + y[4] = 4; // no-warning + y[5] = 5; // expected-warning{{out-of-bound}} + } +} + +void sizeof_vla_2(int a) { + if (a == 5) { + char x[a]; + int y[sizeof(x) / sizeof(char)]; + y[4] = 4; // no-warning + y[5] = 5; // expected-warning{{out-of-bound}} + } +} + +void sizeof_vla_3(int a) { + if (a == 5) { + char x[a]; + int y[sizeof(*&*&*&x)]; + y[4] = 4; // no-warning + y[5] = 5; // expected-warning{{out-of-bound}} + } +} |

