Jordan Rose | bbb6bb4 | 2012-09-08 04:00:03 +0000 | [diff] [blame] | 1 | // RUN: %clang_cc1 -fsyntax-only -verify -Wformat-nonliteral %s |
Ted Kremenek | 1e4c33a | 2010-07-16 02:11:34 +0000 | [diff] [blame] | 2 | |
Hans Wennborg | c876946 | 2012-01-17 09:30:38 +0000 | [diff] [blame] | 3 | // Test that -Wformat=0 works: |
| 4 | // RUN: %clang_cc1 -fsyntax-only -Werror -Wformat=0 %s |
| 5 | |
Hans Wennborg | 439ddaa | 2011-12-12 10:34:18 +0000 | [diff] [blame] | 6 | #include <stdarg.h> |
Ted Kremenek | 1e4c33a | 2010-07-16 02:11:34 +0000 | [diff] [blame] | 7 | typedef __typeof(sizeof(int)) size_t; |
| 8 | typedef struct _FILE FILE; |
Ted Kremenek | 1e51c20 | 2010-07-20 20:04:47 +0000 | [diff] [blame] | 9 | typedef __WCHAR_TYPE__ wchar_t; |
Ted Kremenek | 1e4c33a | 2010-07-16 02:11:34 +0000 | [diff] [blame] | 10 | |
| 11 | int fscanf(FILE * restrict, const char * restrict, ...) ; |
| 12 | int scanf(const char * restrict, ...) ; |
Hans Wennborg | d95a8ab0 | 2011-12-12 18:33:02 +0000 | [diff] [blame] | 13 | int sscanf(const char * restrict, const char * restrict, ...) ; |
Hans Wennborg | 439ddaa | 2011-12-12 10:34:18 +0000 | [diff] [blame] | 14 | int my_scanf(const char * restrict, ...) __attribute__((__format__(__scanf__, 1, 2))); |
| 15 | |
| 16 | int vscanf(const char * restrict, va_list); |
| 17 | int vfscanf(FILE * restrict, const char * restrict, va_list); |
Hans Wennborg | c08e618 | 2011-12-12 18:46:05 +0000 | [diff] [blame] | 18 | int vsscanf(const char * restrict, const char * restrict, va_list); |
Ted Kremenek | 1e4c33a | 2010-07-16 02:11:34 +0000 | [diff] [blame] | 19 | |
| 20 | void test(const char *s, int *i) { |
| 21 | scanf(s, i); // expected-warning{{ormat string is not a string literal}} |
Ted Kremenek | 32d0900 | 2010-07-16 18:27:56 +0000 | [diff] [blame] | 22 | scanf("%0d", i); // expected-warning{{zero field width in scanf format string is unused}} |
| 23 | scanf("%00d", i); // expected-warning{{zero field width in scanf format string is unused}} |
Ted Kremenek | bb09d1e | 2010-07-16 20:49:01 +0000 | [diff] [blame] | 24 | scanf("%d%[asdfasdfd", i, s); // expected-warning{{no closing ']' for '%[' in scanf format string}} |
Ted Kremenek | be86ecc | 2010-07-19 19:47:40 +0000 | [diff] [blame] | 25 | |
| 26 | unsigned short s_x; |
| 27 | scanf ("%" "hu" "\n", &s_x); // no-warning |
Ted Kremenek | c09b6a5 | 2010-07-19 21:25:57 +0000 | [diff] [blame] | 28 | scanf("%y", i); // expected-warning{{invalid conversion specifier 'y'}} |
Ted Kremenek | baa4006 | 2010-07-19 22:01:06 +0000 | [diff] [blame] | 29 | scanf("%%"); // no-warning |
| 30 | scanf("%%%1$d", i); // no-warning |
| 31 | scanf("%1$d%%", i); // no-warning |
| 32 | scanf("%d", i, i); // expected-warning{{data argument not used by format string}} |
| 33 | scanf("%*d", i); // // expected-warning{{data argument not used by format string}} |
| 34 | scanf("%*d", i); // // expected-warning{{data argument not used by format string}} |
| 35 | scanf("%*d%1$d", i); // no-warning |
Hans Wennborg | 58e1e54 | 2012-08-07 08:59:46 +0000 | [diff] [blame] | 36 | |
| 37 | scanf("%s", (char*)0); // no-warning |
| 38 | scanf("%s", (volatile char*)0); // no-warning |
| 39 | scanf("%s", (signed char*)0); // no-warning |
| 40 | scanf("%s", (unsigned char*)0); // no-warning |
| 41 | scanf("%hhu", (signed char*)0); // no-warning |
Ted Kremenek | 1e4c33a | 2010-07-16 02:11:34 +0000 | [diff] [blame] | 42 | } |
Ted Kremenek | 1e51c20 | 2010-07-20 20:04:47 +0000 | [diff] [blame] | 43 | |
| 44 | void bad_length_modifiers(char *s, void *p, wchar_t *ws, long double *ld) { |
| 45 | scanf("%hhs", "foo"); // expected-warning{{length modifier 'hh' results in undefined behavior or no effect with 's' conversion specifier}} |
Hans Wennborg | 6fcd932 | 2011-12-10 13:20:11 +0000 | [diff] [blame] | 46 | scanf("%1$zp", &p); // expected-warning{{length modifier 'z' results in undefined behavior or no effect with 'p' conversion specifier}} |
Ted Kremenek | 1e51c20 | 2010-07-20 20:04:47 +0000 | [diff] [blame] | 47 | scanf("%ls", ws); // no-warning |
| 48 | scanf("%#.2Lf", ld); // expected-warning{{invalid conversion specifier '#'}} |
| 49 | } |
Richard Trieu | 55733de | 2011-10-28 00:41:25 +0000 | [diff] [blame] | 50 | |
| 51 | // Test that the scanf call site is where the warning is attached. If the |
| 52 | // format string is somewhere else, point to it in a note. |
| 53 | void pr9751() { |
| 54 | int *i; |
Hans Wennborg | 6fcd932 | 2011-12-10 13:20:11 +0000 | [diff] [blame] | 55 | char str[100]; |
Richard Trieu | 55733de | 2011-10-28 00:41:25 +0000 | [diff] [blame] | 56 | const char kFormat1[] = "%00d"; // expected-note{{format string is defined here}}} |
| 57 | scanf(kFormat1, i); // expected-warning{{zero field width in scanf format string is unused}} |
| 58 | scanf("%00d", i); // expected-warning{{zero field width in scanf format string is unused}} |
| 59 | const char kFormat2[] = "%["; // expected-note{{format string is defined here}}} |
Hans Wennborg | 6fcd932 | 2011-12-10 13:20:11 +0000 | [diff] [blame] | 60 | scanf(kFormat2, str); // expected-warning{{no closing ']' for '%[' in scanf format string}} |
| 61 | scanf("%[", str); // expected-warning{{no closing ']' for '%[' in scanf format string}} |
Jean-Daniel Dupas | 220947b | 2012-01-31 18:12:08 +0000 | [diff] [blame] | 62 | const char kFormat3[] = "%hu"; // expected-note{{format string is defined here}}} |
| 63 | scanf(kFormat3, &i); // expected-warning {{format specifies type 'unsigned short *' but the argument}} |
| 64 | const char kFormat4[] = "%lp"; // expected-note{{format string is defined here}}} |
| 65 | scanf(kFormat4, &i); // expected-warning {{length modifier 'l' results in undefined behavior or no effect with 'p' conversion specifier}} |
Richard Trieu | 55733de | 2011-10-28 00:41:25 +0000 | [diff] [blame] | 66 | } |
Hans Wennborg | 439ddaa | 2011-12-12 10:34:18 +0000 | [diff] [blame] | 67 | |
| 68 | void test_variants(int *i, const char *s, ...) { |
| 69 | FILE *f = 0; |
| 70 | char buf[100]; |
| 71 | |
Ted Kremenek | ce506ae | 2012-01-20 21:52:58 +0000 | [diff] [blame] | 72 | fscanf(f, "%ld", i); // expected-warning{{format specifies type 'long *' but the argument has type 'int *'}} |
| 73 | sscanf(buf, "%ld", i); // expected-warning{{format specifies type 'long *' but the argument has type 'int *'}} |
| 74 | my_scanf("%ld", i); // expected-warning{{format specifies type 'long *' but the argument has type 'int *'}} |
Hans Wennborg | 439ddaa | 2011-12-12 10:34:18 +0000 | [diff] [blame] | 75 | |
| 76 | va_list ap; |
| 77 | va_start(ap, s); |
| 78 | |
| 79 | vscanf("%[abc", ap); // expected-warning{{no closing ']' for '%[' in scanf format string}} |
| 80 | vfscanf(f, "%[abc", ap); // expected-warning{{no closing ']' for '%[' in scanf format string}} |
| 81 | vsscanf(buf, "%[abc", ap); // expected-warning{{no closing ']' for '%[' in scanf format string}} |
| 82 | } |
Hans Wennborg | d02deeb | 2011-12-15 10:25:47 +0000 | [diff] [blame] | 83 | |
Ted Kremenek | ef1440b | 2012-01-20 22:11:52 +0000 | [diff] [blame] | 84 | void test_scanlist(int *ip, char *sp, wchar_t *ls) { |
Ted Kremenek | ce506ae | 2012-01-20 21:52:58 +0000 | [diff] [blame] | 85 | scanf("%[abc]", ip); // expected-warning{{format specifies type 'char *' but the argument has type 'int *'}} |
Hans Wennborg | 28058d1 | 2012-01-12 15:07:16 +0000 | [diff] [blame] | 86 | scanf("%h[abc]", sp); // expected-warning{{length modifier 'h' results in undefined behavior or no effect with '[' conversion specifier}} |
Ted Kremenek | ef1440b | 2012-01-20 22:11:52 +0000 | [diff] [blame] | 87 | scanf("%l[xyx]", ls); // no-warning |
| 88 | scanf("%ll[xyx]", ls); // expected-warning {{length modifier 'll' results in undefined behavior or no effect with '[' conversion specifier}} |
Hans Wennborg | 6de0b48 | 2012-01-12 14:44:54 +0000 | [diff] [blame] | 89 | } |
| 90 | |
Hans Wennborg | 37969b7 | 2012-01-12 17:11:12 +0000 | [diff] [blame] | 91 | void test_alloc_extension(char **sp, wchar_t **lsp, float *fp) { |
Hans Wennborg | d02deeb | 2011-12-15 10:25:47 +0000 | [diff] [blame] | 92 | /* Make sure "%a" gets parsed as a conversion specifier for float, |
| 93 | * even when followed by an 's', 'S' or '[', which would cause it to be |
| 94 | * parsed as a length modifier in C90. */ |
Ted Kremenek | ce506ae | 2012-01-20 21:52:58 +0000 | [diff] [blame] | 95 | scanf("%as", sp); // expected-warning{{format specifies type 'float *' but the argument has type 'char **'}} |
| 96 | scanf("%aS", lsp); // expected-warning{{format specifies type 'float *' but the argument has type 'wchar_t **'}} |
| 97 | scanf("%a[bcd]", sp); // expected-warning{{format specifies type 'float *' but the argument has type 'char **'}} |
Hans Wennborg | 37969b7 | 2012-01-12 17:11:12 +0000 | [diff] [blame] | 98 | |
| 99 | // Test that the 'm' length modifier is only allowed with s, S, c, C or [. |
| 100 | // TODO: Warn that 'm' is an extension. |
| 101 | scanf("%ms", sp); // No warning. |
| 102 | scanf("%mS", lsp); // No warning. |
| 103 | scanf("%mc", sp); // No warning. |
| 104 | scanf("%mC", lsp); // No warning. |
| 105 | scanf("%m[abc]", sp); // No warning. |
| 106 | scanf("%md", sp); // expected-warning{{length modifier 'm' results in undefined behavior or no effect with 'd' conversion specifier}} |
| 107 | |
| 108 | // Test argument type check for the 'm' length modifier. |
Ted Kremenek | ce506ae | 2012-01-20 21:52:58 +0000 | [diff] [blame] | 109 | scanf("%ms", fp); // expected-warning{{format specifies type 'char **' but the argument has type 'float *'}} |
NAKAMURA Takumi | eb3546e | 2012-09-08 12:06:00 +0000 | [diff] [blame] | 110 | scanf("%mS", fp); // expected-warning-re{{format specifies type 'wchar_t \*\*' \(aka '[^']+'\) but the argument has type 'float \*'}} |
Ted Kremenek | ce506ae | 2012-01-20 21:52:58 +0000 | [diff] [blame] | 111 | scanf("%mc", fp); // expected-warning{{format specifies type 'char **' but the argument has type 'float *'}} |
NAKAMURA Takumi | eb3546e | 2012-09-08 12:06:00 +0000 | [diff] [blame] | 112 | scanf("%mC", fp); // expected-warning-re{{format specifies type 'wchar_t \*\*' \(aka '[^']+'\) but the argument has type 'float \*'}} |
Ted Kremenek | ce506ae | 2012-01-20 21:52:58 +0000 | [diff] [blame] | 113 | scanf("%m[abc]", fp); // expected-warning{{format specifies type 'char **' but the argument has type 'float *'}} |
Hans Wennborg | d02deeb | 2011-12-15 10:25:47 +0000 | [diff] [blame] | 114 | } |
Ted Kremenek | ef1440b | 2012-01-20 22:11:52 +0000 | [diff] [blame] | 115 | |
Hans Wennborg | 32addd5 | 2012-02-16 16:34:54 +0000 | [diff] [blame] | 116 | void test_quad(int *x, long long *llx) { |
| 117 | scanf("%qd", x); // expected-warning{{format specifies type 'long long *' but the argument has type 'int *'}} |
| 118 | scanf("%qd", llx); // no-warning |
| 119 | } |
Hans Wennborg | cec9ce4 | 2012-07-30 17:11:32 +0000 | [diff] [blame] | 120 | |
| 121 | void test_writeback(int *x) { |
| 122 | scanf("%n", (void*)0); // expected-warning{{format specifies type 'int *' but the argument has type 'void *'}} |
| 123 | scanf("%n %c", x, x); // expected-warning{{format specifies type 'char *' but the argument has type 'int *'}} |
Hans Wennborg | f7158fa | 2012-08-07 09:13:19 +0000 | [diff] [blame] | 124 | |
| 125 | scanf("%hhn", (signed char*)0); // no-warning |
| 126 | scanf("%hhn", (char*)0); // no-warning |
| 127 | scanf("%hhn", (unsigned char*)0); // no-warning |
| 128 | scanf("%hhn", (int*)0); // expected-warning{{format specifies type 'signed char *' but the argument has type 'int *'}} |
| 129 | |
| 130 | scanf("%hn", (short*)0); // no-warning |
| 131 | scanf("%hn", (unsigned short*)0); // no-warning |
| 132 | scanf("%hn", (int*)0); // expected-warning{{format specifies type 'short *' but the argument has type 'int *'}} |
| 133 | |
| 134 | scanf("%n", (int*)0); // no-warning |
| 135 | scanf("%n", (unsigned int*)0); // no-warning |
| 136 | scanf("%n", (char*)0); // expected-warning{{format specifies type 'int *' but the argument has type 'char *'}} |
| 137 | |
| 138 | scanf("%ln", (long*)0); // no-warning |
| 139 | scanf("%ln", (unsigned long*)0); // no-warning |
| 140 | scanf("%ln", (int*)0); // expected-warning{{format specifies type 'long *' but the argument has type 'int *'}} |
| 141 | |
| 142 | scanf("%lln", (long long*)0); // no-warning |
| 143 | scanf("%lln", (unsigned long long*)0); // no-warning |
| 144 | scanf("%lln", (int*)0); // expected-warning{{format specifies type 'long long *' but the argument has type 'int *'}} |
| 145 | |
| 146 | scanf("%qn", (long long*)0); // no-warning |
| 147 | scanf("%qn", (unsigned long long*)0); // no-warning |
| 148 | scanf("%qn", (int*)0); // expected-warning{{format specifies type 'long long *' but the argument has type 'int *'}} |
| 149 | |
Hans Wennborg | cec9ce4 | 2012-07-30 17:11:32 +0000 | [diff] [blame] | 150 | } |
Hans Wennborg | 5deddaf | 2012-07-31 16:37:47 +0000 | [diff] [blame] | 151 | |
| 152 | void test_qualifiers(const int *cip, volatile int* vip, |
| 153 | const char *ccp, volatile char* vcp, |
| 154 | const volatile int *cvip) { |
| 155 | scanf("%d", cip); // expected-warning{{format specifies type 'int *' but the argument has type 'const int *'}} |
| 156 | scanf("%n", cip); // expected-warning{{format specifies type 'int *' but the argument has type 'const int *'}} |
| 157 | scanf("%s", ccp); // expected-warning{{format specifies type 'char *' but the argument has type 'const char *'}} |
| 158 | scanf("%d", cvip); // expected-warning{{format specifies type 'int *' but the argument has type 'const volatile int *'}} |
| 159 | |
| 160 | scanf("%d", vip); // No warning. |
| 161 | scanf("%n", vip); // No warning. |
| 162 | scanf("%c", vcp); // No warning. |
| 163 | |
| 164 | typedef int* ip_t; |
| 165 | typedef const int* cip_t; |
| 166 | scanf("%d", (ip_t)0); // No warning. |
| 167 | scanf("%d", (cip_t)0); // expected-warning{{format specifies type 'int *' but the argument has type 'cip_t' (aka 'const int *')}} |
| 168 | } |