blob: 5ce3eb036ebfa7514690158eb9ad7753e7172808 [file] [log] [blame]
Daniel Dunbara5728872009-12-15 20:14:24 +00001// RUN: %clang_cc1 -fsyntax-only -verify -Wformat-nonliteral %s
Chris Lattner59907c42007-08-10 20:18:51 +00002
Chris Lattner59907c42007-08-10 20:18:51 +00003#include <stdarg.h>
Daniel Dunbar23afaad2009-11-17 08:57:36 +00004typedef __typeof(sizeof(int)) size_t;
5typedef struct _FILE FILE;
6int fprintf(FILE *, const char *restrict, ...);
7int printf(const char *restrict, ...);
8int snprintf(char *restrict, size_t, const char *restrict, ...);
9int sprintf(char *restrict, const char *restrict, ...);
10int vasprintf(char **, const char *, va_list);
Chris Lattnere0303582010-01-09 20:43:19 +000011int asprintf(char **, const char *, ...);
Daniel Dunbar23afaad2009-11-17 08:57:36 +000012int vfprintf(FILE *, const char *restrict, va_list);
13int vprintf(const char *restrict, va_list);
14int vsnprintf(char *, size_t, const char *, va_list);
15int vsprintf(char *restrict, const char *restrict, va_list);
Chris Lattner59907c42007-08-10 20:18:51 +000016
Ted Kremenek4a336462007-12-17 19:03:13 +000017char * global_fmt;
18
Chris Lattner59907c42007-08-10 20:18:51 +000019void check_string_literal( FILE* fp, const char* s, char *buf, ... ) {
20
21 char * b;
22 va_list ap;
23 va_start(ap,buf);
24
25 printf(s); // expected-warning {{format string is not a string literal}}
Ted Kremenek4a336462007-12-17 19:03:13 +000026 vprintf(s,ap); // // no-warning
Chris Lattner59907c42007-08-10 20:18:51 +000027 fprintf(fp,s); // expected-warning {{format string is not a string literal}}
Ted Kremenek4a336462007-12-17 19:03:13 +000028 vfprintf(fp,s,ap); // no-warning
Chris Lattner59907c42007-08-10 20:18:51 +000029 asprintf(&b,s); // expected-warning {{format string is not a string lit}}
Ted Kremenek4a336462007-12-17 19:03:13 +000030 vasprintf(&b,s,ap); // no-warning
Chris Lattner59907c42007-08-10 20:18:51 +000031 sprintf(buf,s); // expected-warning {{format string is not a string literal}}
32 snprintf(buf,2,s); // expected-warning {{format string is not a string lit}}
Daniel Dunbarde454282008-10-02 18:44:07 +000033 __builtin___sprintf_chk(buf,0,-1,s); // expected-warning {{format string is not a string literal}}
34 __builtin___snprintf_chk(buf,2,0,-1,s); // expected-warning {{format string is not a string lit}}
Ted Kremenek4a336462007-12-17 19:03:13 +000035 vsprintf(buf,s,ap); // no-warning
36 vsnprintf(buf,2,s,ap); // no-warning
37 vsnprintf(buf,2,global_fmt,ap); // expected-warning {{format string is not a string literal}}
Daniel Dunbarde454282008-10-02 18:44:07 +000038 __builtin___vsnprintf_chk(buf,2,0,-1,s,ap); // no-warning
39 __builtin___vsnprintf_chk(buf,2,0,-1,global_fmt,ap); // expected-warning {{format string is not a string literal}}
Chris Lattner07f192e2009-02-18 18:25:31 +000040
Chris Lattnerd0d082f2009-02-18 18:34:12 +000041 // rdar://6079877
42 printf("abc"
Ted Kremenek31f8e322010-01-29 23:32:22 +000043 "%*d", 1, 1); // no-warning
Chris Lattnerd0d082f2009-02-18 18:34:12 +000044 printf("abc\
45def"
Ted Kremenek31f8e322010-01-29 23:32:22 +000046 "%*d", 1, 1); // no-warning
47
48 // <rdar://problem/6079850>, allow 'unsigned' (instead of 'int') to be used for both
49 // the field width and precision. This deviates from C99, but is reasonably safe
50 // and is also accepted by GCC.
51 printf("%*d", (unsigned) 1, 1); // no-warning
Chris Lattner59907c42007-08-10 20:18:51 +000052}
53
Ted Kremenekd30ef872009-01-12 23:09:09 +000054void check_conditional_literal(const char* s, int i) {
55 printf(i == 1 ? "yes" : "no"); // no-warning
56 printf(i == 0 ? (i == 1 ? "yes" : "no") : "dont know"); // no-warning
Ted Kremenek0fdea462009-01-12 23:09:55 +000057 printf(i == 0 ? (i == 1 ? s : "no") : "dont know"); // expected-warning{{format string is not a string literal}}
Ted Kremenekf88c8e02010-01-29 20:55:36 +000058 printf("yes" ?: "no %d", 1); // expected-warning{{more data arguments than format specifiers}}
Ted Kremenekd30ef872009-01-12 23:09:09 +000059}
60
Ted Kremenek71895b92007-08-14 17:39:48 +000061void check_writeback_specifier()
62{
63 int x;
64 char *b;
65
66 printf("%n",&x); // expected-warning {{'%n' in format string discouraged}}
67 sprintf(b,"%d%%%n",1, &x); // expected-warning {{'%n' in format string dis}}
68}
69
70void check_invalid_specifier(FILE* fp, char *buf)
71{
Ted Kremenekf88c8e02010-01-29 20:55:36 +000072 printf("%s%lb%d","unix",10,20); // expected-warning {{invalid conversion specifier 'b'}}
73 fprintf(fp,"%%%l"); // expected-warning {{incomplete format specifier}}
Ted Kremenekd635c5f2010-01-30 00:49:51 +000074 sprintf(buf,"%%%%%ld%d%d", 1, 2, 3); // expected-warning{{conversion specifies type 'long' but the argument has type 'int'}}
75 snprintf(buf, 2, "%%%%%ld%;%d", 1, 2, 3); // expected-warning{{conversion specifies type 'long' but the argument has type 'int'}} expected-warning {{invalid conversion specifier ';'}}
Ted Kremenek71895b92007-08-14 17:39:48 +000076}
77
78void check_null_char_string(char* b)
79{
80 printf("\0this is bogus%d",1); // expected-warning {{string contains '\0'}}
81 snprintf(b,10,"%%%%%d\0%d",1,2); // expected-warning {{string contains '\0'}}
82 printf("%\0d",1); // expected-warning {{string contains '\0'}}
83}
84
Anders Carlsson6eda8c92007-10-12 17:48:41 +000085void check_empty_format_string(char* buf, ...)
Ted Kremenek71895b92007-08-14 17:39:48 +000086{
87 va_list ap;
88 va_start(ap,buf);
89 vprintf("",ap); // expected-warning {{format string is empty}}
90 sprintf(buf,""); // expected-warning {{format string is empty}}
91}
92
Anders Carlsson6eda8c92007-10-12 17:48:41 +000093void check_wide_string(char* b, ...)
Ted Kremenek71895b92007-08-14 17:39:48 +000094{
Ted Kremenek71895b92007-08-14 17:39:48 +000095 va_list ap;
96 va_start(ap,b);
97
Daniel Dunbar4489fe12008-08-05 00:07:51 +000098 printf(L"foo %d",2); // expected-warning {{incompatible pointer types}}, expected-warning {{should not be a wide string}}
John Thompsonaddb0f62009-10-29 00:10:42 +000099 vsprintf(b,L"bar %d",ap); // expected-warning {{incompatible pointer types}}, expected-warning {{should not be a wide string}}
Ted Kremenek71895b92007-08-14 17:39:48 +0000100}
Ted Kremenek580b6642007-10-12 20:51:52 +0000101
102void check_asterisk_precision_width(int x) {
103 printf("%*d"); // expected-warning {{'*' specified field width is missing a matching 'int' argument}}
104 printf("%.*d"); // expected-warning {{'.*' specified field precision is missing a matching 'int' argument}}
105 printf("%*d",12,x); // no-warning
106 printf("%*d","foo",x); // expected-warning {{field width should have type 'int', but argument has type 'char *'}}
107 printf("%.*d","foo",x); // expected-warning {{field precision should have type 'int', but argument has type 'char *'}}
Daniel Dunbar4489fe12008-08-05 00:07:51 +0000108}
Douglas Gregor3c385e52009-02-14 18:57:46 +0000109
110void __attribute__((format(printf,1,3))) myprintf(const char*, int blah, ...);
111
112void test_myprintf() {
113 myprintf("%d", 17, 18); // okay
114}
Ted Kremenek082d9362009-03-20 21:35:28 +0000115
116void test_constant_bindings(void) {
117 const char * const s1 = "hello";
118 const char s2[] = "hello";
119 const char *s3 = "hello";
120 char * const s4 = "hello";
121 extern const char s5[];
122
123 printf(s1); // no-warning
124 printf(s2); // no-warning
125 printf(s3); // expected-warning{{not a string literal}}
126 printf(s4); // expected-warning{{not a string literal}}
127 printf(s5); // expected-warning{{not a string literal}}
128}
Chris Lattner655f1412009-04-29 04:59:47 +0000129
130
131// Test what happens when -Wformat-security only.
132#pragma GCC diagnostic ignored "-Wformat-nonliteral"
133#pragma GCC diagnostic warning "-Wformat-security"
134
135void test9(char *P) {
136 int x;
137 printf(P); // expected-warning {{format string is not a string literal (potentially insecure)}}
138 printf(P, 42);
139 printf("%n", &x); // expected-warning {{use of '%n' in format string discouraged }}
140}
Ted Kremenek42ae3e82009-05-13 16:06:05 +0000141
142void torture(va_list v8) {
143 vprintf ("%*.*d", v8); // no-warning
Ted Kremenek31f8e322010-01-29 23:32:22 +0000144
Ted Kremenek42ae3e82009-05-13 16:06:05 +0000145}
146
Daniel Dunbar01aefc62010-01-30 15:49:20 +0000147void test10(int x, float f, int i, long long lli) {
Ted Kremenekf88c8e02010-01-29 20:55:36 +0000148 printf("%@", 12); // expected-warning{{invalid conversion specifier '@'}}
149 printf("\0"); // expected-warning{{format string contains '\0' within the string body}}
150 printf("xs\0"); // expected-warning{{format string contains '\0' within the string body}}
151 printf("%*d\n"); // expected-warning{{'*' specified field width is missing a matching 'int' argument}}
152 printf("%*.*d\n", x); // expected-warning{{'.*' specified field precision is missing a matching 'int' argument}}
153 printf("%*d\n", f, x); // expected-warning{{field width should have type 'int', but argument has type 'double'}}
154 printf("%*.*d\n", x, f, x); // expected-warning{{field precision should have type 'int', but argument has type 'double'}}
155 printf("%**\n"); // expected-warning{{invalid conversion specifier '*'}}
156 printf("%n", &i); // expected-warning{{use of '%n' in format string discouraged (potentially insecure)}}
157 printf("%d%d\n", x); // expected-warning{{more '%' conversions than data arguments}}
158 printf("%d\n", x, x); // expected-warning{{more data arguments than format specifiers}}
159 printf("%W%d%Z\n", x, x, x); // expected-warning{{invalid conversion specifier 'W'}} expected-warning{{invalid conversion specifier 'Z'}}
160 printf("%"); // expected-warning{{incomplete format specifier}}
161 printf("%.d", x); // no-warning
162 printf("%.", x); // expected-warning{{incomplete format specifier}}
Ted Kremenekc9a89fe2010-01-30 01:02:18 +0000163 printf("%f", 4); // expected-warning{{conversion specifies type 'double' but the argument has type 'int'}}
Daniel Dunbar01aefc62010-01-30 15:49:20 +0000164 printf("%qd", lli);
Ted Kremenek180f2842010-02-01 19:28:15 +0000165 printf("hhX %hhX", (unsigned char)10); // no-warning
166 printf("llX %llX", (long long) 10); // no-warning
167 // This is fine, because there is an implicit conversion to an int.
168 printf("%d", (unsigned char) 10); // no-warning
169 printf("%d", (long long) 10); // expected-warning{{conversion specifies type 'int' but the argument has type 'long long'}}
Ted Kremenekf911eba2010-02-01 23:23:50 +0000170 printf("%Lf\n", (long double) 1.0); // no-warning
171 printf("%f\n", (long double) 1.0); // expected-warning{{conversion specifies type 'double' but the argument has type 'long double'}}
Ted Kremenekf88c8e02010-01-29 20:55:36 +0000172}
173
Ted Kremenekd635c5f2010-01-30 00:49:51 +0000174typedef struct __aslclient *aslclient;
175typedef struct __aslmsg *aslmsg;
176int asl_log(aslclient asl, aslmsg msg, int level, const char *format, ...) __attribute__((__format__ (__printf__, 4, 5)));
177void test_asl(aslclient asl) {
178 // Test case from <rdar://problem/7341605>.
179 asl_log(asl, 0, 3, "Error: %m"); // no-warning
180 asl_log(asl, 0, 3, "Error: %W"); // expected-warning{{invalid conversion specifier 'W'}}
181}
Douglas Gregoraa74a1e2010-02-02 20:10:50 +0000182
183// <rdar://problem/7595366>
184typedef enum { A } int_t;
185void f0(int_t x) { printf("%d\n", x); }