diff options
author | Anna Zaks <ganna@apple.com> | 2012-01-31 19:33:39 +0000 |
---|---|---|
committer | Anna Zaks <ganna@apple.com> | 2012-01-31 19:33:39 +0000 |
commit | e00575f12cf280621ef0ed4d69e909bdfc9fef62 (patch) | |
tree | 2b3e9d85103550c967dd4c577ad6cccc4730a599 /test/Analysis/cstring-syntax.c | |
parent | 393b9793da0b62e26e3974c88a0bca18f2d7fd5e (diff) |
[analyzer] Add checks for common anti-patterns in strncat.
(Since this is syntax only, might be a good candidate for turning into a
compiler warning.)
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@149407 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'test/Analysis/cstring-syntax.c')
-rw-r--r-- | test/Analysis/cstring-syntax.c | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/test/Analysis/cstring-syntax.c b/test/Analysis/cstring-syntax.c new file mode 100644 index 0000000000..b6e898bcc8 --- /dev/null +++ b/test/Analysis/cstring-syntax.c @@ -0,0 +1,13 @@ +// RUN: %clang_cc1 -analyze -analyzer-checker=experimental.unix.cstring.BadSizeArg -analyzer-store=region -Wno-strlcpy-strlcat-size -Wno-sizeof-array-argument -Wno-sizeof-pointer-memaccess -verify %s + +typedef __SIZE_TYPE__ size_t; +char *strncat(char *, const char *, size_t); +size_t strlen (const char *s); + +void testStrncat(const char *src) { + char dest[10]; + strncat(dest, "AAAAAAAAAAAAAAAAAAAAAAAAAAAAA", sizeof(dest) - 1); // expected-warning {{Potential buffer overflow. Replace with 'sizeof(dest) - strlen(dest) - 1' or use a safer 'strlcat' API}} + strncat(dest, "AAAAAAAAAAAAAAAAAAAAAAAAAAA", sizeof(dest)); // expected-warning {{Potential buffer overflow. Replace with}} + strncat(dest, "AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA", sizeof(dest) - strlen(dest)); // expected-warning {{Potential buffer overflow. Replace with}} + strncat(dest, src, sizeof(src)); // expected-warning {{Potential buffer overflow. Replace with}} +} |