blob: 3b64508b5a6893ddc88482f9f1e4be17a63edc6c [file] [log] [blame]
Ted Kremenek02087932010-07-16 02:11:22 +00001//== PrintfFormatString.cpp - Analysis of printf format strings --*- C++ -*-==//
Ted Kremeneka2e77b42010-01-27 23:43:25 +00002//
3// The LLVM Compiler Infrastructure
4//
5// This file is distributed under the University of Illinois Open Source
6// License. See LICENSE.TXT for details.
7//
8//===----------------------------------------------------------------------===//
9//
10// Handling of format string in printf and friends. The structure of format
11// strings for fprintf() are described in C99 7.19.6.1.
12//
13//===----------------------------------------------------------------------===//
14
Ted Kremenek02087932010-07-16 02:11:22 +000015#include "clang/Analysis/Analyses/FormatString.h"
16#include "FormatStringParsing.h"
Chandler Carruth3a022472012-12-04 09:13:33 +000017#include "clang/Basic/TargetInfo.h"
Ted Kremeneka2e77b42010-01-27 23:43:25 +000018
Hans Wennborgc3b3da02012-08-07 08:11:26 +000019using clang::analyze_format_string::ArgType;
Ted Kremenek02087932010-07-16 02:11:22 +000020using clang::analyze_format_string::FormatStringHandler;
21using clang::analyze_format_string::LengthModifier;
22using clang::analyze_format_string::OptionalAmount;
Ted Kremenekf03e6d852010-07-20 20:04:27 +000023using clang::analyze_format_string::ConversionSpecifier;
Ted Kremenek02087932010-07-16 02:11:22 +000024using clang::analyze_printf::PrintfSpecifier;
Ted Kremenekd1668192010-02-27 01:41:03 +000025
Ted Kremenekc22f78d2010-01-29 03:16:21 +000026using namespace clang;
Ted Kremeneka2e77b42010-01-27 23:43:25 +000027
Ted Kremenek02087932010-07-16 02:11:22 +000028typedef clang::analyze_format_string::SpecifierResult<PrintfSpecifier>
29 PrintfSpecifierResult;
Ted Kremenekc8b188d2010-02-16 01:46:59 +000030
31//===----------------------------------------------------------------------===//
32// Methods for parsing format strings.
33//===----------------------------------------------------------------------===//
Ted Kremeneka2e77b42010-01-27 23:43:25 +000034
Ted Kremenek02087932010-07-16 02:11:22 +000035using analyze_format_string::ParseNonPositionAmount;
Ted Kremenekc8b188d2010-02-16 01:46:59 +000036
Ted Kremenek02087932010-07-16 02:11:22 +000037static bool ParsePrecision(FormatStringHandler &H, PrintfSpecifier &FS,
Ted Kremenekd1668192010-02-27 01:41:03 +000038 const char *Start, const char *&Beg, const char *E,
39 unsigned *argIndex) {
40 if (argIndex) {
41 FS.setPrecision(ParseNonPositionAmount(Beg, E, *argIndex));
Chad Rosier6fdf38b2011-08-17 23:08:45 +000042 } else {
Ted Kremenekd1668192010-02-27 01:41:03 +000043 const OptionalAmount Amt = ParsePositionAmount(H, Start, Beg, E,
Ted Kremenek02087932010-07-16 02:11:22 +000044 analyze_format_string::PrecisionPos);
Ted Kremenekd1668192010-02-27 01:41:03 +000045 if (Amt.isInvalid())
46 return true;
47 FS.setPrecision(Amt);
48 }
49 return false;
50}
51
Ted Kremenek2b417712015-07-02 05:39:16 +000052static bool ParseObjCFlags(FormatStringHandler &H, PrintfSpecifier &FS,
53 const char *FlagBeg, const char *E, bool Warn) {
54 StringRef Flag(FlagBeg, E - FlagBeg);
55 // Currently there is only one flag.
56 if (Flag == "tt") {
57 FS.setHasObjCTechnicalTerm(FlagBeg);
58 return false;
59 }
60 // Handle either the case of no flag or an invalid flag.
61 if (Warn) {
62 if (Flag == "")
63 H.HandleEmptyObjCModifierFlag(FlagBeg, E - FlagBeg);
64 else
65 H.HandleInvalidObjCModifierFlag(FlagBeg, E - FlagBeg);
66 }
67 return true;
68}
69
Ted Kremenek02087932010-07-16 02:11:22 +000070static PrintfSpecifierResult ParsePrintfSpecifier(FormatStringHandler &H,
Ted Kremenek1de17072010-02-04 20:46:58 +000071 const char *&Beg,
Ted Kremenek4a49d982010-02-26 19:18:41 +000072 const char *E,
Hans Wennborg23926bd2011-12-15 10:25:47 +000073 unsigned &argIndex,
Jordan Rose510260c2012-09-13 02:11:03 +000074 const LangOptions &LO,
Fariborz Jahanian6485fe42014-09-09 23:10:54 +000075 const TargetInfo &Target,
Dimitry Andric6b5ed342015-02-19 22:32:33 +000076 bool Warn,
77 bool isFreeBSDKPrintf) {
Ted Kremenekc8b188d2010-02-16 01:46:59 +000078
Ted Kremenekf03e6d852010-07-20 20:04:27 +000079 using namespace clang::analyze_format_string;
Ted Kremenek176f7d62010-01-29 02:13:53 +000080 using namespace clang::analyze_printf;
Ted Kremenekc8b188d2010-02-16 01:46:59 +000081
Ted Kremeneka2e77b42010-01-27 23:43:25 +000082 const char *I = Beg;
Craig Topper25542942014-05-20 04:30:07 +000083 const char *Start = nullptr;
Ted Kremeneka2e77b42010-01-27 23:43:25 +000084 UpdateOnReturn <const char*> UpdateBeg(Beg, I);
85
86 // Look for a '%' character that indicates the start of a format specifier.
Ted Kremenekb5c98ef2010-01-28 23:56:52 +000087 for ( ; I != E ; ++I) {
Ted Kremeneka2e77b42010-01-27 23:43:25 +000088 char c = *I;
Ted Kremeneka2e77b42010-01-27 23:43:25 +000089 if (c == '\0') {
90 // Detect spurious null characters, which are likely errors.
91 H.HandleNullChar(I);
92 return true;
93 }
94 if (c == '%') {
Ted Kremenekb5c98ef2010-01-28 23:56:52 +000095 Start = I++; // Record the start of the format specifier.
Ted Kremeneka2e77b42010-01-27 23:43:25 +000096 break;
97 }
98 }
Ted Kremenekc8b188d2010-02-16 01:46:59 +000099
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000100 // No format specifier found?
101 if (!Start)
102 return false;
Ted Kremenekc8b188d2010-02-16 01:46:59 +0000103
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000104 if (I == E) {
105 // No more characters left?
Fariborz Jahanian6485fe42014-09-09 23:10:54 +0000106 if (Warn)
107 H.HandleIncompleteSpecifier(Start, E - Start);
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000108 return true;
109 }
Ted Kremenekc8b188d2010-02-16 01:46:59 +0000110
Ted Kremenek02087932010-07-16 02:11:22 +0000111 PrintfSpecifier FS;
Ted Kremenekd1668192010-02-27 01:41:03 +0000112 if (ParseArgPosition(H, FS, Start, I, E))
113 return true;
114
115 if (I == E) {
116 // No more characters left?
Fariborz Jahanian6485fe42014-09-09 23:10:54 +0000117 if (Warn)
118 H.HandleIncompleteSpecifier(Start, E - Start);
Ted Kremenekd1668192010-02-27 01:41:03 +0000119 return true;
120 }
Ted Kremenekc8b188d2010-02-16 01:46:59 +0000121
Mehdi Amini06d367c2016-10-24 20:39:34 +0000122 const char *OSLogVisibilityFlagsStart = nullptr,
123 *OSLogVisibilityFlagsEnd = nullptr;
124 if (*I == '{') {
125 OSLogVisibilityFlagsStart = I++;
126 // Find the end of the modifier.
127 while (I != E && *I != '}') {
128 I++;
129 }
130 if (I == E) {
131 if (Warn)
132 H.HandleIncompleteSpecifier(Start, E - Start);
133 return true;
134 }
135 assert(*I == '}');
136 OSLogVisibilityFlagsEnd = I++;
137
138 // Just see if 'private' or 'public' is the first word. os_log itself will
139 // do any further parsing.
140 const char *P = OSLogVisibilityFlagsStart + 1;
141 while (P < OSLogVisibilityFlagsEnd && isspace(*P))
142 P++;
143 const char *WordStart = P;
144 while (P < OSLogVisibilityFlagsEnd && (isalnum(*P) || *P == '_'))
145 P++;
146 const char *WordEnd = P;
147 StringRef Word(WordStart, WordEnd - WordStart);
148 if (Word == "private") {
149 FS.setIsPrivate(WordStart);
150 } else if (Word == "public") {
151 FS.setIsPublic(WordStart);
152 }
153 }
154
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000155 // Look for flags (if any).
156 bool hasMore = true;
157 for ( ; I != E; ++I) {
158 switch (*I) {
159 default: hasMore = false; break;
NAKAMURA Takumif9cbcc42011-01-27 07:10:08 +0000160 case '\'':
Ted Kremeneka322cae2011-01-08 05:28:38 +0000161 // FIXME: POSIX specific. Always accept?
162 FS.setHasThousandsGrouping(I);
163 break;
Tom Careb49ec692010-06-17 19:00:27 +0000164 case '-': FS.setIsLeftJustified(I); break;
165 case '+': FS.setHasPlusPrefix(I); break;
166 case ' ': FS.setHasSpacePrefix(I); break;
167 case '#': FS.setHasAlternativeForm(I); break;
168 case '0': FS.setHasLeadingZeros(I); break;
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000169 }
170 if (!hasMore)
171 break;
Ted Kremenekc8b188d2010-02-16 01:46:59 +0000172 }
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000173
174 if (I == E) {
175 // No more characters left?
Fariborz Jahanian6485fe42014-09-09 23:10:54 +0000176 if (Warn)
177 H.HandleIncompleteSpecifier(Start, E - Start);
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000178 return true;
179 }
Ted Kremenekc8b188d2010-02-16 01:46:59 +0000180
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000181 // Look for the field width (if any).
Ted Kremenekd1668192010-02-27 01:41:03 +0000182 if (ParseFieldWidth(H, FS, Start, I, E,
Craig Topper25542942014-05-20 04:30:07 +0000183 FS.usesPositionalArg() ? nullptr : &argIndex))
Ted Kremenekd1668192010-02-27 01:41:03 +0000184 return true;
Ted Kremenekc8b188d2010-02-16 01:46:59 +0000185
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000186 if (I == E) {
187 // No more characters left?
Fariborz Jahanian6485fe42014-09-09 23:10:54 +0000188 if (Warn)
189 H.HandleIncompleteSpecifier(Start, E - Start);
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000190 return true;
Ted Kremenekc8b188d2010-02-16 01:46:59 +0000191 }
192
193 // Look for the precision (if any).
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000194 if (*I == '.') {
Ted Kremenekc22f78d2010-01-29 03:16:21 +0000195 ++I;
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000196 if (I == E) {
Fariborz Jahanian6485fe42014-09-09 23:10:54 +0000197 if (Warn)
198 H.HandleIncompleteSpecifier(Start, E - Start);
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000199 return true;
200 }
Ted Kremenekc8b188d2010-02-16 01:46:59 +0000201
Ted Kremenekd1668192010-02-27 01:41:03 +0000202 if (ParsePrecision(H, FS, Start, I, E,
Craig Topper25542942014-05-20 04:30:07 +0000203 FS.usesPositionalArg() ? nullptr : &argIndex))
Ted Kremenekd1668192010-02-27 01:41:03 +0000204 return true;
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000205
206 if (I == E) {
207 // No more characters left?
Fariborz Jahanian6485fe42014-09-09 23:10:54 +0000208 if (Warn)
209 H.HandleIncompleteSpecifier(Start, E - Start);
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000210 return true;
211 }
212 }
213
214 // Look for the length modifier.
Hans Wennborg23926bd2011-12-15 10:25:47 +0000215 if (ParseLengthModifier(FS, I, E, LO) && I == E) {
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000216 // No more characters left?
Fariborz Jahanian6485fe42014-09-09 23:10:54 +0000217 if (Warn)
218 H.HandleIncompleteSpecifier(Start, E - Start);
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000219 return true;
220 }
Ted Kremenek2a0cd592010-02-09 00:04:09 +0000221
Ted Kremenek2b417712015-07-02 05:39:16 +0000222 // Look for the Objective-C modifier flags, if any.
223 // We parse these here, even if they don't apply to
224 // the conversion specifier, and then emit an error
225 // later if the conversion specifier isn't '@'. This
226 // enables better recovery, and we don't know if
227 // these flags are applicable until later.
228 const char *ObjCModifierFlagsStart = nullptr,
229 *ObjCModifierFlagsEnd = nullptr;
230 if (*I == '[') {
231 ObjCModifierFlagsStart = I;
232 ++I;
233 auto flagStart = I;
234 for (;; ++I) {
235 ObjCModifierFlagsEnd = I;
236 if (I == E) {
237 if (Warn)
238 H.HandleIncompleteSpecifier(Start, E - Start);
239 return true;
240 }
241 // Did we find the closing ']'?
242 if (*I == ']') {
243 if (ParseObjCFlags(H, FS, flagStart, I, Warn))
244 return true;
245 ++I;
246 break;
247 }
248 // There are no separators defined yet for multiple
249 // Objective-C modifier flags. When those are
250 // defined, this is the place to check.
251 }
252 }
253
Ted Kremenek23a71a12010-01-29 20:29:53 +0000254 if (*I == '\0') {
Ted Kremenek2a0cd592010-02-09 00:04:09 +0000255 // Detect spurious null characters, which are likely errors.
256 H.HandleNullChar(I);
257 return true;
Ted Kremenek23a71a12010-01-29 20:29:53 +0000258 }
Ted Kremenekc8b188d2010-02-16 01:46:59 +0000259
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000260 // Finally, look for the conversion specifier.
Ted Kremenekfee0e962010-01-28 02:46:17 +0000261 const char *conversionPosition = I++;
Ted Kremenek94af5752010-01-29 02:40:24 +0000262 ConversionSpecifier::Kind k = ConversionSpecifier::InvalidSpecifier;
Ted Kremenekfee0e962010-01-28 02:46:17 +0000263 switch (*conversionPosition) {
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000264 default:
Ted Kremenek94af5752010-01-29 02:40:24 +0000265 break;
Ted Kremenekc06ead62010-01-28 00:55:28 +0000266 // C99: 7.19.6.1 (section 8).
Ted Kremenek74a4ce72010-02-24 00:05:54 +0000267 case '%': k = ConversionSpecifier::PercentArg; break;
268 case 'A': k = ConversionSpecifier::AArg; break;
269 case 'E': k = ConversionSpecifier::EArg; break;
270 case 'F': k = ConversionSpecifier::FArg; break;
271 case 'G': k = ConversionSpecifier::GArg; break;
272 case 'X': k = ConversionSpecifier::XArg; break;
273 case 'a': k = ConversionSpecifier::aArg; break;
Ted Kremenekab0fe8a2010-07-20 20:03:49 +0000274 case 'c': k = ConversionSpecifier::cArg; break;
Ted Kremenekfee0e962010-01-28 02:46:17 +0000275 case 'd': k = ConversionSpecifier::dArg; break;
Ted Kremenek74a4ce72010-02-24 00:05:54 +0000276 case 'e': k = ConversionSpecifier::eArg; break;
277 case 'f': k = ConversionSpecifier::fArg; break;
278 case 'g': k = ConversionSpecifier::gArg; break;
Ted Kremenekfee0e962010-01-28 02:46:17 +0000279 case 'i': k = ConversionSpecifier::iArg; break;
Ted Kremenek516ef222010-07-20 20:04:10 +0000280 case 'n': k = ConversionSpecifier::nArg; break;
Ted Kremenekfee0e962010-01-28 02:46:17 +0000281 case 'o': k = ConversionSpecifier::oArg; break;
David Majnemer3cba4952013-08-21 21:54:46 +0000282 case 'p': k = ConversionSpecifier::pArg; break;
283 case 's': k = ConversionSpecifier::sArg; break;
Ted Kremenekfee0e962010-01-28 02:46:17 +0000284 case 'u': k = ConversionSpecifier::uArg; break;
285 case 'x': k = ConversionSpecifier::xArg; break;
Ted Kremeneka322cae2011-01-08 05:28:38 +0000286 // POSIX specific.
Ted Kremenek74a4ce72010-02-24 00:05:54 +0000287 case 'C': k = ConversionSpecifier::CArg; break;
Ted Kremenek348d1492010-07-20 20:04:15 +0000288 case 'S': k = ConversionSpecifier::SArg; break;
Mehdi Amini06d367c2016-10-24 20:39:34 +0000289 // Apple extension for os_log
290 case 'P':
291 k = ConversionSpecifier::PArg;
292 break;
Ted Kremenekc06ead62010-01-28 00:55:28 +0000293 // Objective-C.
Ted Kremenek23a71a12010-01-29 20:29:53 +0000294 case '@': k = ConversionSpecifier::ObjCObjArg; break;
Ted Kremenek2a0cd592010-02-09 00:04:09 +0000295 // Glibc specific.
Ted Kremenek23a71a12010-01-29 20:29:53 +0000296 case 'm': k = ConversionSpecifier::PrintErrno; break;
Dimitry Andric6b5ed342015-02-19 22:32:33 +0000297 // FreeBSD kernel specific.
298 case 'b':
299 if (isFreeBSDKPrintf)
300 k = ConversionSpecifier::FreeBSDbArg; // int followed by char *
301 break;
302 case 'r':
303 if (isFreeBSDKPrintf)
304 k = ConversionSpecifier::FreeBSDrArg; // int
305 break;
306 case 'y':
307 if (isFreeBSDKPrintf)
308 k = ConversionSpecifier::FreeBSDyArg; // int
309 break;
Hans Wennborgc597b4c2014-09-07 03:03:51 +0000310 // Apple-specific.
Jordan Rose510260c2012-09-13 02:11:03 +0000311 case 'D':
Dimitry Andric6b5ed342015-02-19 22:32:33 +0000312 if (isFreeBSDKPrintf)
313 k = ConversionSpecifier::FreeBSDDArg; // void * followed by char *
314 else if (Target.getTriple().isOSDarwin())
Jordan Rose510260c2012-09-13 02:11:03 +0000315 k = ConversionSpecifier::DArg;
316 break;
317 case 'O':
318 if (Target.getTriple().isOSDarwin())
319 k = ConversionSpecifier::OArg;
320 break;
321 case 'U':
322 if (Target.getTriple().isOSDarwin())
323 k = ConversionSpecifier::UArg;
324 break;
Hans Wennborgc597b4c2014-09-07 03:03:51 +0000325 // MS specific.
326 case 'Z':
327 if (Target.getTriple().isOSMSVCRT())
328 k = ConversionSpecifier::ZArg;
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000329 }
Ted Kremenek2b417712015-07-02 05:39:16 +0000330
331 // Check to see if we used the Objective-C modifier flags with
332 // a conversion specifier other than '@'.
333 if (k != ConversionSpecifier::ObjCObjArg &&
334 k != ConversionSpecifier::InvalidSpecifier &&
335 ObjCModifierFlagsStart) {
336 H.HandleObjCFlagsWithNonObjCConversion(ObjCModifierFlagsStart,
337 ObjCModifierFlagsEnd + 1,
338 conversionPosition);
339 return true;
340 }
Mehdi Amini06d367c2016-10-24 20:39:34 +0000341
Ted Kremenekf03e6d852010-07-20 20:04:27 +0000342 PrintfConversionSpecifier CS(conversionPosition, k);
Ted Kremenek4a49d982010-02-26 19:18:41 +0000343 FS.setConversionSpecifier(CS);
Ted Kremenekd1668192010-02-27 01:41:03 +0000344 if (CS.consumesDataArgument() && !FS.usesPositionalArg())
Ted Kremenek4a49d982010-02-26 19:18:41 +0000345 FS.setArgIndex(argIndex++);
Dimitry Andric6b5ed342015-02-19 22:32:33 +0000346 // FreeBSD kernel specific.
347 if (k == ConversionSpecifier::FreeBSDbArg ||
348 k == ConversionSpecifier::FreeBSDDArg)
349 argIndex++;
Ted Kremenek94af5752010-01-29 02:40:24 +0000350
351 if (k == ConversionSpecifier::InvalidSpecifier) {
Bruno Cardoso Lopes0c18d032016-03-29 17:35:02 +0000352 unsigned Len = I - Start;
353 if (ParseUTF8InvalidSpecifier(Start, E, Len)) {
354 CS.setEndScanList(Start + Len);
355 FS.setConversionSpecifier(CS);
356 }
Ted Kremenek4a49d982010-02-26 19:18:41 +0000357 // Assume the conversion takes one argument.
Bruno Cardoso Lopes0c18d032016-03-29 17:35:02 +0000358 return !H.HandleInvalidPrintfConversionSpecifier(FS, Start, Len);
Ted Kremenek94af5752010-01-29 02:40:24 +0000359 }
Ted Kremenek02087932010-07-16 02:11:22 +0000360 return PrintfSpecifierResult(Start, FS);
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000361}
362
Ted Kremenek02087932010-07-16 02:11:22 +0000363bool clang::analyze_format_string::ParsePrintfString(FormatStringHandler &H,
364 const char *I,
Hans Wennborg23926bd2011-12-15 10:25:47 +0000365 const char *E,
Jordan Rose510260c2012-09-13 02:11:03 +0000366 const LangOptions &LO,
Dimitry Andric6b5ed342015-02-19 22:32:33 +0000367 const TargetInfo &Target,
368 bool isFreeBSDKPrintf) {
Ted Kremenek4a49d982010-02-26 19:18:41 +0000369
370 unsigned argIndex = 0;
371
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000372 // Keep looking for a format specifier until we have exhausted the string.
373 while (I != E) {
Hans Wennborg23926bd2011-12-15 10:25:47 +0000374 const PrintfSpecifierResult &FSR = ParsePrintfSpecifier(H, I, E, argIndex,
Dimitry Andric6b5ed342015-02-19 22:32:33 +0000375 LO, Target, true,
376 isFreeBSDKPrintf);
Ted Kremenek94af5752010-01-29 02:40:24 +0000377 // Did a fail-stop error of any kind occur when parsing the specifier?
378 // If so, don't do any more processing.
379 if (FSR.shouldStop())
Dmitri Gribenko76bb5cabfa2012-09-10 21:20:09 +0000380 return true;
Ted Kremenek94af5752010-01-29 02:40:24 +0000381 // Did we exhaust the string or encounter an error that
382 // we can recover from?
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000383 if (!FSR.hasValue())
Ted Kremenek94af5752010-01-29 02:40:24 +0000384 continue;
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000385 // We have a format specifier. Pass it to the callback.
Ted Kremenek02087932010-07-16 02:11:22 +0000386 if (!H.HandlePrintfSpecifier(FSR.getValue(), FSR.getStart(),
Ted Kremenekfee0e962010-01-28 02:46:17 +0000387 I - FSR.getStart()))
Ted Kremenek23a71a12010-01-29 20:29:53 +0000388 return true;
Ted Kremenekc8b188d2010-02-16 01:46:59 +0000389 }
390 assert(I == E && "Format string not exhausted");
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000391 return false;
392}
Ted Kremeneka2e77b42010-01-27 23:43:25 +0000393
Fariborz Jahanian6485fe42014-09-09 23:10:54 +0000394bool clang::analyze_format_string::ParseFormatStringHasSArg(const char *I,
395 const char *E,
396 const LangOptions &LO,
397 const TargetInfo &Target) {
398
399 unsigned argIndex = 0;
400
401 // Keep looking for a %s format specifier until we have exhausted the string.
402 FormatStringHandler H;
403 while (I != E) {
404 const PrintfSpecifierResult &FSR = ParsePrintfSpecifier(H, I, E, argIndex,
Dimitry Andric6b5ed342015-02-19 22:32:33 +0000405 LO, Target, false,
406 false);
Fariborz Jahanian6485fe42014-09-09 23:10:54 +0000407 // Did a fail-stop error of any kind occur when parsing the specifier?
408 // If so, don't do any more processing.
409 if (FSR.shouldStop())
410 return false;
411 // Did we exhaust the string or encounter an error that
412 // we can recover from?
413 if (!FSR.hasValue())
414 continue;
415 const analyze_printf::PrintfSpecifier &FS = FSR.getValue();
416 // Return true if this a %s format specifier.
417 if (FS.getConversionSpecifier().getKind() == ConversionSpecifier::Kind::sArg)
418 return true;
419 }
420 return false;
421}
422
Ted Kremenekc8b188d2010-02-16 01:46:59 +0000423//===----------------------------------------------------------------------===//
Ted Kremenek02087932010-07-16 02:11:22 +0000424// Methods on PrintfSpecifier.
Tom Careb7042702010-06-09 04:11:11 +0000425//===----------------------------------------------------------------------===//
426
Hans Wennborgc3b3da02012-08-07 08:11:26 +0000427ArgType PrintfSpecifier::getArgType(ASTContext &Ctx,
428 bool IsObjCLiteral) const {
Ted Kremenekf03e6d852010-07-20 20:04:27 +0000429 const PrintfConversionSpecifier &CS = getConversionSpecifier();
NAKAMURA Takumif9cbcc42011-01-27 07:10:08 +0000430
Ted Kremenek79db7b72010-01-29 22:59:32 +0000431 if (!CS.consumesDataArgument())
Hans Wennborgc3b3da02012-08-07 08:11:26 +0000432 return ArgType::Invalid();
Ted Kremenekc8b188d2010-02-16 01:46:59 +0000433
Ted Kremenek5f0c0662010-08-24 22:24:51 +0000434 if (CS.getKind() == ConversionSpecifier::cArg)
435 switch (LM.getKind()) {
Hans Wennborg1b231582014-09-04 21:39:52 +0000436 case LengthModifier::None:
437 return Ctx.IntTy;
Hans Wennborga5b1aa92011-12-09 12:22:12 +0000438 case LengthModifier::AsLong:
Hans Wennborg68f42b92014-09-04 21:39:46 +0000439 case LengthModifier::AsWide:
Hans Wennborgc3b3da02012-08-07 08:11:26 +0000440 return ArgType(ArgType::WIntTy, "wint_t");
Hans Wennborg1b231582014-09-04 21:39:52 +0000441 case LengthModifier::AsShort:
442 if (Ctx.getTargetInfo().getTriple().isOSMSVCRT())
443 return Ctx.IntTy;
Galina Kistanovade7e2212017-06-03 06:23:51 +0000444 LLVM_FALLTHROUGH;
Ted Kremenek5f0c0662010-08-24 22:24:51 +0000445 default:
Hans Wennborgc3b3da02012-08-07 08:11:26 +0000446 return ArgType::Invalid();
Ted Kremenek5f0c0662010-08-24 22:24:51 +0000447 }
NAKAMURA Takumif9cbcc42011-01-27 07:10:08 +0000448
Ted Kremenek79db7b72010-01-29 22:59:32 +0000449 if (CS.isIntArg())
Tom Careb7042702010-06-09 04:11:11 +0000450 switch (LM.getKind()) {
451 case LengthModifier::AsLongDouble:
Ted Kremenek6fa57272012-01-24 21:29:54 +0000452 // GNU extension.
453 return Ctx.LongLongTy;
David Majnemer3cba4952013-08-21 21:54:46 +0000454 case LengthModifier::None:
455 return Ctx.IntTy;
456 case LengthModifier::AsInt32:
457 return ArgType(Ctx.IntTy, "__int32");
Hans Wennborgc3b3da02012-08-07 08:11:26 +0000458 case LengthModifier::AsChar: return ArgType::AnyCharTy;
Tom Careb7042702010-06-09 04:11:11 +0000459 case LengthModifier::AsShort: return Ctx.ShortTy;
460 case LengthModifier::AsLong: return Ctx.LongTy;
Hans Wennborg9bc9bcc2012-02-16 16:34:54 +0000461 case LengthModifier::AsLongLong:
462 case LengthModifier::AsQuad:
463 return Ctx.LongLongTy;
David Majnemer3cba4952013-08-21 21:54:46 +0000464 case LengthModifier::AsInt64:
465 return ArgType(Ctx.LongLongTy, "__int64");
Hans Wennborg772e9272011-12-07 10:33:11 +0000466 case LengthModifier::AsIntMax:
Hans Wennborgc3b3da02012-08-07 08:11:26 +0000467 return ArgType(Ctx.getIntMaxType(), "intmax_t");
Hans Wennborg27541db2011-10-27 08:29:09 +0000468 case LengthModifier::AsSizeT:
Alexander Shaposhnikov441c1d92017-07-14 22:57:00 +0000469 return ArgType(Ctx.getSignedSizeType(), "ssize_t");
David Majnemer3cba4952013-08-21 21:54:46 +0000470 case LengthModifier::AsInt3264:
David Majnemera39da8e2013-08-22 07:53:21 +0000471 return Ctx.getTargetInfo().getTriple().isArch64Bit()
472 ? ArgType(Ctx.LongLongTy, "__int64")
473 : ArgType(Ctx.IntTy, "__int32");
Hans Wennborg772e9272011-12-07 10:33:11 +0000474 case LengthModifier::AsPtrDiff:
Hans Wennborgc3b3da02012-08-07 08:11:26 +0000475 return ArgType(Ctx.getPointerDiffType(), "ptrdiff_t");
Hans Wennborg23926bd2011-12-15 10:25:47 +0000476 case LengthModifier::AsAllocate:
Hans Wennborg6073e312012-01-12 17:11:12 +0000477 case LengthModifier::AsMAllocate:
Hans Wennborg68f42b92014-09-04 21:39:46 +0000478 case LengthModifier::AsWide:
Hans Wennborgc3b3da02012-08-07 08:11:26 +0000479 return ArgType::Invalid();
Ted Kremenek79db7b72010-01-29 22:59:32 +0000480 }
481
482 if (CS.isUIntArg())
Tom Careb7042702010-06-09 04:11:11 +0000483 switch (LM.getKind()) {
484 case LengthModifier::AsLongDouble:
Ted Kremenek6fa57272012-01-24 21:29:54 +0000485 // GNU extension.
486 return Ctx.UnsignedLongLongTy;
David Majnemer3cba4952013-08-21 21:54:46 +0000487 case LengthModifier::None:
488 return Ctx.UnsignedIntTy;
489 case LengthModifier::AsInt32:
490 return ArgType(Ctx.UnsignedIntTy, "unsigned __int32");
Tom Careb7042702010-06-09 04:11:11 +0000491 case LengthModifier::AsChar: return Ctx.UnsignedCharTy;
492 case LengthModifier::AsShort: return Ctx.UnsignedShortTy;
493 case LengthModifier::AsLong: return Ctx.UnsignedLongTy;
Hans Wennborg9bc9bcc2012-02-16 16:34:54 +0000494 case LengthModifier::AsLongLong:
495 case LengthModifier::AsQuad:
496 return Ctx.UnsignedLongLongTy;
David Majnemer3cba4952013-08-21 21:54:46 +0000497 case LengthModifier::AsInt64:
498 return ArgType(Ctx.UnsignedLongLongTy, "unsigned __int64");
Hans Wennborg772e9272011-12-07 10:33:11 +0000499 case LengthModifier::AsIntMax:
Hans Wennborgc3b3da02012-08-07 08:11:26 +0000500 return ArgType(Ctx.getUIntMaxType(), "uintmax_t");
Tom Careb7042702010-06-09 04:11:11 +0000501 case LengthModifier::AsSizeT:
Hans Wennborgc3b3da02012-08-07 08:11:26 +0000502 return ArgType(Ctx.getSizeType(), "size_t");
David Majnemer3cba4952013-08-21 21:54:46 +0000503 case LengthModifier::AsInt3264:
504 return Ctx.getTargetInfo().getTriple().isArch64Bit()
David Majnemera39da8e2013-08-22 07:53:21 +0000505 ? ArgType(Ctx.UnsignedLongLongTy, "unsigned __int64")
506 : ArgType(Ctx.UnsignedIntTy, "unsigned __int32");
Tom Careb7042702010-06-09 04:11:11 +0000507 case LengthModifier::AsPtrDiff:
Alexander Shaposhnikov195b25c2017-09-28 23:11:31 +0000508 return ArgType(Ctx.getUnsignedPointerDiffType(), "unsigned ptrdiff_t");
Hans Wennborg23926bd2011-12-15 10:25:47 +0000509 case LengthModifier::AsAllocate:
Hans Wennborg6073e312012-01-12 17:11:12 +0000510 case LengthModifier::AsMAllocate:
Hans Wennborg68f42b92014-09-04 21:39:46 +0000511 case LengthModifier::AsWide:
Hans Wennborgc3b3da02012-08-07 08:11:26 +0000512 return ArgType::Invalid();
Ted Kremenek79db7b72010-01-29 22:59:32 +0000513 }
Ted Kremenekc8b188d2010-02-16 01:46:59 +0000514
Ted Kremenek016b6052010-02-01 23:23:50 +0000515 if (CS.isDoubleArg()) {
Tom Careb7042702010-06-09 04:11:11 +0000516 if (LM.getKind() == LengthModifier::AsLongDouble)
Ted Kremenek016b6052010-02-01 23:23:50 +0000517 return Ctx.LongDoubleTy;
Ted Kremenek9ff02052010-01-30 01:02:18 +0000518 return Ctx.DoubleTy;
Ted Kremenek016b6052010-02-01 23:23:50 +0000519 }
Ted Kremenek4a49d982010-02-26 19:18:41 +0000520
Hans Wennborgabc1e222012-08-07 09:13:19 +0000521 if (CS.getKind() == ConversionSpecifier::nArg) {
522 switch (LM.getKind()) {
523 case LengthModifier::None:
524 return ArgType::PtrTo(Ctx.IntTy);
525 case LengthModifier::AsChar:
526 return ArgType::PtrTo(Ctx.SignedCharTy);
527 case LengthModifier::AsShort:
528 return ArgType::PtrTo(Ctx.ShortTy);
529 case LengthModifier::AsLong:
530 return ArgType::PtrTo(Ctx.LongTy);
531 case LengthModifier::AsLongLong:
532 case LengthModifier::AsQuad:
533 return ArgType::PtrTo(Ctx.LongLongTy);
534 case LengthModifier::AsIntMax:
535 return ArgType::PtrTo(ArgType(Ctx.getIntMaxType(), "intmax_t"));
536 case LengthModifier::AsSizeT:
Alexander Shaposhnikov441c1d92017-07-14 22:57:00 +0000537 return ArgType::PtrTo(ArgType(Ctx.getSignedSizeType(), "ssize_t"));
Hans Wennborgabc1e222012-08-07 09:13:19 +0000538 case LengthModifier::AsPtrDiff:
539 return ArgType::PtrTo(ArgType(Ctx.getPointerDiffType(), "ptrdiff_t"));
540 case LengthModifier::AsLongDouble:
541 return ArgType(); // FIXME: Is this a known extension?
542 case LengthModifier::AsAllocate:
543 case LengthModifier::AsMAllocate:
David Majnemer3cba4952013-08-21 21:54:46 +0000544 case LengthModifier::AsInt32:
545 case LengthModifier::AsInt3264:
546 case LengthModifier::AsInt64:
Hans Wennborg68f42b92014-09-04 21:39:46 +0000547 case LengthModifier::AsWide:
Hans Wennborgabc1e222012-08-07 09:13:19 +0000548 return ArgType::Invalid();
549 }
550 }
551
Ted Kremenek74a4ce72010-02-24 00:05:54 +0000552 switch (CS.getKind()) {
Hans Wennborg0c353262011-12-09 15:57:33 +0000553 case ConversionSpecifier::sArg:
Nico Weber496cdc22012-01-31 01:43:25 +0000554 if (LM.getKind() == LengthModifier::AsWideChar) {
555 if (IsObjCLiteral)
Jordan Rose0e5badd2012-12-05 18:44:49 +0000556 return ArgType(Ctx.getPointerType(Ctx.UnsignedShortTy.withConst()),
557 "const unichar *");
Hans Wennborgc3b3da02012-08-07 08:11:26 +0000558 return ArgType(ArgType::WCStrTy, "wchar_t *");
Nico Weber496cdc22012-01-31 01:43:25 +0000559 }
Hans Wennborg68f42b92014-09-04 21:39:46 +0000560 if (LM.getKind() == LengthModifier::AsWide)
561 return ArgType(ArgType::WCStrTy, "wchar_t *");
Hans Wennborgc3b3da02012-08-07 08:11:26 +0000562 return ArgType::CStrTy;
Ted Kremenek348d1492010-07-20 20:04:15 +0000563 case ConversionSpecifier::SArg:
Nico Weber496cdc22012-01-31 01:43:25 +0000564 if (IsObjCLiteral)
Jordan Rose0e5badd2012-12-05 18:44:49 +0000565 return ArgType(Ctx.getPointerType(Ctx.UnsignedShortTy.withConst()),
566 "const unichar *");
Hans Wennborg1b231582014-09-04 21:39:52 +0000567 if (Ctx.getTargetInfo().getTriple().isOSMSVCRT() &&
568 LM.getKind() == LengthModifier::AsShort)
569 return ArgType::CStrTy;
Hans Wennborgc3b3da02012-08-07 08:11:26 +0000570 return ArgType(ArgType::WCStrTy, "wchar_t *");
Ted Kremenek74a4ce72010-02-24 00:05:54 +0000571 case ConversionSpecifier::CArg:
Nico Weber496cdc22012-01-31 01:43:25 +0000572 if (IsObjCLiteral)
Jordan Rose0e5badd2012-12-05 18:44:49 +0000573 return ArgType(Ctx.UnsignedShortTy, "unichar");
Hans Wennborg1b231582014-09-04 21:39:52 +0000574 if (Ctx.getTargetInfo().getTriple().isOSMSVCRT() &&
575 LM.getKind() == LengthModifier::AsShort)
576 return Ctx.IntTy;
Hans Wennborg0d81e012013-05-10 10:08:40 +0000577 return ArgType(Ctx.WideCharTy, "wchar_t");
Ted Kremenek1ce32be2010-07-20 20:04:04 +0000578 case ConversionSpecifier::pArg:
Mehdi Amini06d367c2016-10-24 20:39:34 +0000579 case ConversionSpecifier::PArg:
Hans Wennborgc3b3da02012-08-07 08:11:26 +0000580 return ArgType::CPointerTy;
Ted Kremenek9b1f3d42012-01-25 00:04:09 +0000581 case ConversionSpecifier::ObjCObjArg:
Hans Wennborgc3b3da02012-08-07 08:11:26 +0000582 return ArgType::ObjCPointerTy;
Ted Kremenek74a4ce72010-02-24 00:05:54 +0000583 default:
584 break;
585 }
Ted Kremenek4a49d982010-02-26 19:18:41 +0000586
Ted Kremenek79db7b72010-01-29 22:59:32 +0000587 // FIXME: Handle other cases.
Hans Wennborgc3b3da02012-08-07 08:11:26 +0000588 return ArgType();
Ted Kremenek79db7b72010-01-29 22:59:32 +0000589}
590
Hans Wennborgd99d6882012-02-15 09:59:46 +0000591bool PrintfSpecifier::fixType(QualType QT, const LangOptions &LangOpt,
592 ASTContext &Ctx, bool IsObjCLiteral) {
Hans Wennborgebcd1c72012-07-30 17:11:32 +0000593 // %n is different from other conversion specifiers; don't try to fix it.
594 if (CS.getKind() == ConversionSpecifier::nArg)
595 return false;
596
Jordan Rose68f6d3b2012-05-30 21:53:13 +0000597 // Handle Objective-C objects first. Note that while the '%@' specifier will
598 // not warn for structure pointer or void pointer arguments (because that's
599 // how CoreFoundation objects are implemented), we only show a fixit for '%@'
600 // if we know it's an object (block, id, class, or __attribute__((NSObject))).
601 if (QT->isObjCRetainableType()) {
602 if (!IsObjCLiteral)
603 return false;
604
605 CS.setKind(ConversionSpecifier::ObjCObjArg);
606
607 // Disable irrelevant flags
608 HasThousandsGrouping = false;
609 HasPlusPrefix = false;
610 HasSpacePrefix = false;
611 HasAlternativeForm = false;
612 HasLeadingZeroes = false;
613 Precision.setHowSpecified(OptionalAmount::NotSpecified);
614 LM.setKind(LengthModifier::None);
615
616 return true;
617 }
618
619 // Handle strings next (char *, wchar_t *)
Tom Careb7042702010-06-09 04:11:11 +0000620 if (QT->isPointerType() && (QT->getPointeeType()->isAnyCharacterType())) {
Ted Kremenek563e3ed2010-07-20 20:03:58 +0000621 CS.setKind(ConversionSpecifier::sArg);
Tom Careb7042702010-06-09 04:11:11 +0000622
Tom Care00554632010-06-11 04:22:02 +0000623 // Disable irrelevant flags
624 HasAlternativeForm = 0;
625 HasLeadingZeroes = 0;
626
Tom Careb7042702010-06-09 04:11:11 +0000627 // Set the long length modifier for wide characters
628 if (QT->getPointeeType()->isWideCharType())
629 LM.setKind(LengthModifier::AsWideChar);
Hans Wennborg2027de32011-12-09 10:51:29 +0000630 else
631 LM.setKind(LengthModifier::None);
Tom Careb7042702010-06-09 04:11:11 +0000632
633 return true;
634 }
635
Jordan Rose614e72b2012-06-04 22:49:02 +0000636 // If it's an enum, get its underlying type.
637 if (const EnumType *ETy = QT->getAs<EnumType>())
638 QT = ETy->getDecl()->getIntegerType();
639
Tom Careb7042702010-06-09 04:11:11 +0000640 // We can only work with builtin types.
Tom Careb7042702010-06-09 04:11:11 +0000641 const BuiltinType *BT = QT->getAs<BuiltinType>();
John McCall59207972011-10-18 22:28:37 +0000642 if (!BT)
643 return false;
Tom Care00554632010-06-11 04:22:02 +0000644
Tom Careb7042702010-06-09 04:11:11 +0000645 // Set length modifier
646 switch (BT->getKind()) {
Eli Friedman7ec652a2011-04-27 22:06:20 +0000647 case BuiltinType::Bool:
648 case BuiltinType::WChar_U:
649 case BuiltinType::WChar_S:
Richard Smith3a8244d2018-05-01 05:02:45 +0000650 case BuiltinType::Char8: // FIXME: Treat like 'char'?
Eli Friedman7ec652a2011-04-27 22:06:20 +0000651 case BuiltinType::Char16:
652 case BuiltinType::Char32:
653 case BuiltinType::UInt128:
654 case BuiltinType::Int128:
Anton Korobeynikovf0c267e2011-10-14 23:23:15 +0000655 case BuiltinType::Half:
Sjoerd Meijercc623ad2017-09-08 15:15:00 +0000656 case BuiltinType::Float16:
Nemanja Ivanovicbb1ea2d2016-05-09 08:52:33 +0000657 case BuiltinType::Float128:
Leonard Chanf921d852018-06-04 16:07:52 +0000658 case BuiltinType::ShortAccum:
659 case BuiltinType::Accum:
660 case BuiltinType::LongAccum:
661 case BuiltinType::UShortAccum:
662 case BuiltinType::UAccum:
663 case BuiltinType::ULongAccum:
Leonard Chanab80f3c2018-06-14 14:53:51 +0000664 case BuiltinType::ShortFract:
665 case BuiltinType::Fract:
666 case BuiltinType::LongFract:
667 case BuiltinType::UShortFract:
668 case BuiltinType::UFract:
669 case BuiltinType::ULongFract:
670 case BuiltinType::SatShortAccum:
671 case BuiltinType::SatAccum:
672 case BuiltinType::SatLongAccum:
673 case BuiltinType::SatUShortAccum:
674 case BuiltinType::SatUAccum:
675 case BuiltinType::SatULongAccum:
676 case BuiltinType::SatShortFract:
677 case BuiltinType::SatFract:
678 case BuiltinType::SatLongFract:
679 case BuiltinType::SatUShortFract:
680 case BuiltinType::SatUFract:
681 case BuiltinType::SatULongFract:
John McCall59207972011-10-18 22:28:37 +0000682 // Various types which are non-trivial to correct.
Eli Friedman7ec652a2011-04-27 22:06:20 +0000683 return false;
684
Alexey Bader954ba212016-04-08 13:40:33 +0000685#define IMAGE_TYPE(ImgType, Id, SingletonId, Access, Suffix) \
686 case BuiltinType::Id:
Alexey Baderb62f1442016-04-13 08:33:41 +0000687#include "clang/Basic/OpenCLImageTypes.def"
John McCall59207972011-10-18 22:28:37 +0000688#define SIGNED_TYPE(Id, SingletonId)
689#define UNSIGNED_TYPE(Id, SingletonId)
690#define FLOATING_TYPE(Id, SingletonId)
691#define BUILTIN_TYPE(Id, SingletonId) \
692 case BuiltinType::Id:
693#include "clang/AST/BuiltinTypes.def"
Eli Friedman7ec652a2011-04-27 22:06:20 +0000694 // Misc other stuff which doesn't make sense here.
695 return false;
696
697 case BuiltinType::UInt:
698 case BuiltinType::Int:
699 case BuiltinType::Float:
700 case BuiltinType::Double:
Tom Care00554632010-06-11 04:22:02 +0000701 LM.setKind(LengthModifier::None);
Tom Careb7042702010-06-09 04:11:11 +0000702 break;
Tom Care00554632010-06-11 04:22:02 +0000703
Ted Kremenek12a37de2010-10-21 04:00:58 +0000704 case BuiltinType::Char_U:
705 case BuiltinType::UChar:
706 case BuiltinType::Char_S:
707 case BuiltinType::SChar:
708 LM.setKind(LengthModifier::AsChar);
709 break;
710
711 case BuiltinType::Short:
712 case BuiltinType::UShort:
713 LM.setKind(LengthModifier::AsShort);
714 break;
715
Tom Careb7042702010-06-09 04:11:11 +0000716 case BuiltinType::Long:
717 case BuiltinType::ULong:
718 LM.setKind(LengthModifier::AsLong);
719 break;
720
721 case BuiltinType::LongLong:
722 case BuiltinType::ULongLong:
723 LM.setKind(LengthModifier::AsLongLong);
724 break;
725
726 case BuiltinType::LongDouble:
727 LM.setKind(LengthModifier::AsLongDouble);
728 break;
729 }
730
Hans Wennborgf99d04f2011-10-18 08:10:06 +0000731 // Handle size_t, ptrdiff_t, etc. that have dedicated length modifiers in C99.
Richard Smith2bf7fdb2013-01-02 11:42:31 +0000732 if (isa<TypedefType>(QT) && (LangOpt.C99 || LangOpt.CPlusPlus11))
Hans Wennborg08574d32012-07-27 19:17:46 +0000733 namedTypeToLengthModifier(QT, LM);
Hans Wennborgf99d04f2011-10-18 08:10:06 +0000734
Jordan Roseaa7a3b32013-01-17 18:47:16 +0000735 // If fixing the length modifier was enough, we might be done.
Jordan Rose92303592012-09-08 04:00:03 +0000736 if (hasValidLengthModifier(Ctx.getTargetInfo())) {
Jordan Roseaa7a3b32013-01-17 18:47:16 +0000737 // If we're going to offer a fix anyway, make sure the sign matches.
738 switch (CS.getKind()) {
739 case ConversionSpecifier::uArg:
740 case ConversionSpecifier::UArg:
741 if (QT->isSignedIntegerType())
742 CS.setKind(clang::analyze_format_string::ConversionSpecifier::dArg);
743 break;
744 case ConversionSpecifier::dArg:
745 case ConversionSpecifier::DArg:
746 case ConversionSpecifier::iArg:
Jordan Rose1eb34292013-01-17 22:34:10 +0000747 if (QT->isUnsignedIntegerType() && !HasPlusPrefix)
Jordan Roseaa7a3b32013-01-17 18:47:16 +0000748 CS.setKind(clang::analyze_format_string::ConversionSpecifier::uArg);
749 break;
750 default:
751 // Other specifiers do not have signed/unsigned variants.
752 break;
753 }
754
Jordan Rose92303592012-09-08 04:00:03 +0000755 const analyze_printf::ArgType &ATR = getArgType(Ctx, IsObjCLiteral);
756 if (ATR.isValid() && ATR.matchesType(Ctx, QT))
757 return true;
758 }
Hans Wennborgd99d6882012-02-15 09:59:46 +0000759
Tom Careb7042702010-06-09 04:11:11 +0000760 // Set conversion specifier and disable any flags which do not apply to it.
Ted Kremenek12a37de2010-10-21 04:00:58 +0000761 // Let typedefs to char fall through to int, as %c is silly for uint8_t.
Jordan Rose6aaa87e2012-12-05 18:44:37 +0000762 if (!isa<TypedefType>(QT) && QT->isCharType()) {
Ted Kremenekab0fe8a2010-07-20 20:03:49 +0000763 CS.setKind(ConversionSpecifier::cArg);
Ted Kremenek12a37de2010-10-21 04:00:58 +0000764 LM.setKind(LengthModifier::None);
Tom Careb7042702010-06-09 04:11:11 +0000765 Precision.setHowSpecified(OptionalAmount::NotSpecified);
766 HasAlternativeForm = 0;
767 HasLeadingZeroes = 0;
Tom Careb49ec692010-06-17 19:00:27 +0000768 HasPlusPrefix = 0;
Tom Careb7042702010-06-09 04:11:11 +0000769 }
770 // Test for Floating type first as LongDouble can pass isUnsignedIntegerType
Douglas Gregor49b4d732010-06-22 23:07:26 +0000771 else if (QT->isRealFloatingType()) {
Tom Careb7042702010-06-09 04:11:11 +0000772 CS.setKind(ConversionSpecifier::fArg);
773 }
Tom Careb7042702010-06-09 04:11:11 +0000774 else if (QT->isSignedIntegerType()) {
775 CS.setKind(ConversionSpecifier::dArg);
776 HasAlternativeForm = 0;
777 }
Douglas Gregorb4e3533ce2010-06-09 05:25:34 +0000778 else if (QT->isUnsignedIntegerType()) {
Hans Wennborgd99d6882012-02-15 09:59:46 +0000779 CS.setKind(ConversionSpecifier::uArg);
Tom Careb7042702010-06-09 04:11:11 +0000780 HasAlternativeForm = 0;
Tom Careb49ec692010-06-17 19:00:27 +0000781 HasPlusPrefix = 0;
Chad Rosier6fdf38b2011-08-17 23:08:45 +0000782 } else {
David Blaikie83d382b2011-09-23 05:06:16 +0000783 llvm_unreachable("Unexpected type");
Tom Careb7042702010-06-09 04:11:11 +0000784 }
785
786 return true;
787}
788
Chris Lattner0e62c1c2011-07-23 10:55:15 +0000789void PrintfSpecifier::toString(raw_ostream &os) const {
Tom Careb7042702010-06-09 04:11:11 +0000790 // Whilst some features have no defined order, we are using the order
NAKAMURA Takumi7c288862011-01-27 07:09:49 +0000791 // appearing in the C99 standard (ISO/IEC 9899:1999 (E) 7.19.6.1)
Tom Careb7042702010-06-09 04:11:11 +0000792 os << "%";
793
794 // Positional args
795 if (usesPositionalArg()) {
796 os << getPositionalArgIndex() << "$";
797 }
798
799 // Conversion flags
800 if (IsLeftJustified) os << "-";
801 if (HasPlusPrefix) os << "+";
802 if (HasSpacePrefix) os << " ";
803 if (HasAlternativeForm) os << "#";
804 if (HasLeadingZeroes) os << "0";
805
806 // Minimum field width
807 FieldWidth.toString(os);
808 // Precision
809 Precision.toString(os);
810 // Length modifier
811 os << LM.toString();
812 // Conversion specifier
813 os << CS.toString();
814}
Tom Careb49ec692010-06-17 19:00:27 +0000815
Ted Kremenek02087932010-07-16 02:11:22 +0000816bool PrintfSpecifier::hasValidPlusPrefix() const {
Tom Careb49ec692010-06-17 19:00:27 +0000817 if (!HasPlusPrefix)
818 return true;
819
820 // The plus prefix only makes sense for signed conversions
821 switch (CS.getKind()) {
822 case ConversionSpecifier::dArg:
Jordan Rose510260c2012-09-13 02:11:03 +0000823 case ConversionSpecifier::DArg:
Tom Careb49ec692010-06-17 19:00:27 +0000824 case ConversionSpecifier::iArg:
825 case ConversionSpecifier::fArg:
826 case ConversionSpecifier::FArg:
827 case ConversionSpecifier::eArg:
828 case ConversionSpecifier::EArg:
829 case ConversionSpecifier::gArg:
830 case ConversionSpecifier::GArg:
831 case ConversionSpecifier::aArg:
832 case ConversionSpecifier::AArg:
Dimitry Andric6b5ed342015-02-19 22:32:33 +0000833 case ConversionSpecifier::FreeBSDrArg:
834 case ConversionSpecifier::FreeBSDyArg:
Tom Careb49ec692010-06-17 19:00:27 +0000835 return true;
836
837 default:
838 return false;
839 }
840}
841
Ted Kremenek02087932010-07-16 02:11:22 +0000842bool PrintfSpecifier::hasValidAlternativeForm() const {
Tom Careb49ec692010-06-17 19:00:27 +0000843 if (!HasAlternativeForm)
844 return true;
845
Anders Carlssona369f8d2010-11-21 18:34:21 +0000846 // Alternate form flag only valid with the oxXaAeEfFgG conversions
Tom Careb49ec692010-06-17 19:00:27 +0000847 switch (CS.getKind()) {
848 case ConversionSpecifier::oArg:
Jordan Rose510260c2012-09-13 02:11:03 +0000849 case ConversionSpecifier::OArg:
Tom Careb49ec692010-06-17 19:00:27 +0000850 case ConversionSpecifier::xArg:
Anders Carlssona369f8d2010-11-21 18:34:21 +0000851 case ConversionSpecifier::XArg:
Tom Careb49ec692010-06-17 19:00:27 +0000852 case ConversionSpecifier::aArg:
853 case ConversionSpecifier::AArg:
854 case ConversionSpecifier::eArg:
855 case ConversionSpecifier::EArg:
856 case ConversionSpecifier::fArg:
857 case ConversionSpecifier::FArg:
858 case ConversionSpecifier::gArg:
859 case ConversionSpecifier::GArg:
Dimitry Andric6b5ed342015-02-19 22:32:33 +0000860 case ConversionSpecifier::FreeBSDrArg:
861 case ConversionSpecifier::FreeBSDyArg:
Tom Careb49ec692010-06-17 19:00:27 +0000862 return true;
863
864 default:
865 return false;
866 }
867}
868
Ted Kremenek02087932010-07-16 02:11:22 +0000869bool PrintfSpecifier::hasValidLeadingZeros() const {
Tom Careb49ec692010-06-17 19:00:27 +0000870 if (!HasLeadingZeroes)
871 return true;
872
873 // Leading zeroes flag only valid with the diouxXaAeEfFgG conversions
874 switch (CS.getKind()) {
875 case ConversionSpecifier::dArg:
Jordan Rose510260c2012-09-13 02:11:03 +0000876 case ConversionSpecifier::DArg:
Tom Careb49ec692010-06-17 19:00:27 +0000877 case ConversionSpecifier::iArg:
878 case ConversionSpecifier::oArg:
Jordan Rose510260c2012-09-13 02:11:03 +0000879 case ConversionSpecifier::OArg:
Tom Careb49ec692010-06-17 19:00:27 +0000880 case ConversionSpecifier::uArg:
Jordan Rose510260c2012-09-13 02:11:03 +0000881 case ConversionSpecifier::UArg:
Tom Careb49ec692010-06-17 19:00:27 +0000882 case ConversionSpecifier::xArg:
883 case ConversionSpecifier::XArg:
884 case ConversionSpecifier::aArg:
885 case ConversionSpecifier::AArg:
886 case ConversionSpecifier::eArg:
887 case ConversionSpecifier::EArg:
888 case ConversionSpecifier::fArg:
889 case ConversionSpecifier::FArg:
890 case ConversionSpecifier::gArg:
891 case ConversionSpecifier::GArg:
Dimitry Andric6b5ed342015-02-19 22:32:33 +0000892 case ConversionSpecifier::FreeBSDrArg:
893 case ConversionSpecifier::FreeBSDyArg:
Tom Careb49ec692010-06-17 19:00:27 +0000894 return true;
895
896 default:
897 return false;
898 }
899}
900
Ted Kremenek02087932010-07-16 02:11:22 +0000901bool PrintfSpecifier::hasValidSpacePrefix() const {
Tom Careb49ec692010-06-17 19:00:27 +0000902 if (!HasSpacePrefix)
903 return true;
904
905 // The space prefix only makes sense for signed conversions
906 switch (CS.getKind()) {
907 case ConversionSpecifier::dArg:
Jordan Rose510260c2012-09-13 02:11:03 +0000908 case ConversionSpecifier::DArg:
Tom Careb49ec692010-06-17 19:00:27 +0000909 case ConversionSpecifier::iArg:
910 case ConversionSpecifier::fArg:
911 case ConversionSpecifier::FArg:
912 case ConversionSpecifier::eArg:
913 case ConversionSpecifier::EArg:
914 case ConversionSpecifier::gArg:
915 case ConversionSpecifier::GArg:
916 case ConversionSpecifier::aArg:
917 case ConversionSpecifier::AArg:
Dimitry Andric6b5ed342015-02-19 22:32:33 +0000918 case ConversionSpecifier::FreeBSDrArg:
919 case ConversionSpecifier::FreeBSDyArg:
Tom Careb49ec692010-06-17 19:00:27 +0000920 return true;
921
922 default:
923 return false;
924 }
925}
926
Ted Kremenek02087932010-07-16 02:11:22 +0000927bool PrintfSpecifier::hasValidLeftJustified() const {
Tom Careb49ec692010-06-17 19:00:27 +0000928 if (!IsLeftJustified)
929 return true;
930
931 // The left justified flag is valid for all conversions except n
932 switch (CS.getKind()) {
Ted Kremenek516ef222010-07-20 20:04:10 +0000933 case ConversionSpecifier::nArg:
Tom Careb49ec692010-06-17 19:00:27 +0000934 return false;
935
936 default:
937 return true;
938 }
939}
940
Ted Kremenekbf4832c2011-01-08 05:28:46 +0000941bool PrintfSpecifier::hasValidThousandsGroupingPrefix() const {
942 if (!HasThousandsGrouping)
943 return true;
NAKAMURA Takumif9cbcc42011-01-27 07:10:08 +0000944
Ted Kremenekbf4832c2011-01-08 05:28:46 +0000945 switch (CS.getKind()) {
946 case ConversionSpecifier::dArg:
Jordan Rose510260c2012-09-13 02:11:03 +0000947 case ConversionSpecifier::DArg:
Ted Kremenekbf4832c2011-01-08 05:28:46 +0000948 case ConversionSpecifier::iArg:
949 case ConversionSpecifier::uArg:
Jordan Rose510260c2012-09-13 02:11:03 +0000950 case ConversionSpecifier::UArg:
Ted Kremenekbf4832c2011-01-08 05:28:46 +0000951 case ConversionSpecifier::fArg:
952 case ConversionSpecifier::FArg:
953 case ConversionSpecifier::gArg:
954 case ConversionSpecifier::GArg:
955 return true;
956 default:
957 return false;
958 }
959}
960
Ted Kremenek02087932010-07-16 02:11:22 +0000961bool PrintfSpecifier::hasValidPrecision() const {
Tom Careb49ec692010-06-17 19:00:27 +0000962 if (Precision.getHowSpecified() == OptionalAmount::NotSpecified)
963 return true;
964
Mehdi Amini06d367c2016-10-24 20:39:34 +0000965 // Precision is only valid with the diouxXaAeEfFgGsP conversions
Tom Careb49ec692010-06-17 19:00:27 +0000966 switch (CS.getKind()) {
967 case ConversionSpecifier::dArg:
Jordan Rose510260c2012-09-13 02:11:03 +0000968 case ConversionSpecifier::DArg:
Tom Careb49ec692010-06-17 19:00:27 +0000969 case ConversionSpecifier::iArg:
970 case ConversionSpecifier::oArg:
Jordan Rose510260c2012-09-13 02:11:03 +0000971 case ConversionSpecifier::OArg:
Tom Careb49ec692010-06-17 19:00:27 +0000972 case ConversionSpecifier::uArg:
Jordan Rose510260c2012-09-13 02:11:03 +0000973 case ConversionSpecifier::UArg:
Tom Careb49ec692010-06-17 19:00:27 +0000974 case ConversionSpecifier::xArg:
975 case ConversionSpecifier::XArg:
976 case ConversionSpecifier::aArg:
977 case ConversionSpecifier::AArg:
978 case ConversionSpecifier::eArg:
979 case ConversionSpecifier::EArg:
980 case ConversionSpecifier::fArg:
981 case ConversionSpecifier::FArg:
982 case ConversionSpecifier::gArg:
983 case ConversionSpecifier::GArg:
Ted Kremenek563e3ed2010-07-20 20:03:58 +0000984 case ConversionSpecifier::sArg:
Dimitry Andric6b5ed342015-02-19 22:32:33 +0000985 case ConversionSpecifier::FreeBSDrArg:
986 case ConversionSpecifier::FreeBSDyArg:
Mehdi Amini06d367c2016-10-24 20:39:34 +0000987 case ConversionSpecifier::PArg:
Tom Careb49ec692010-06-17 19:00:27 +0000988 return true;
989
990 default:
991 return false;
992 }
993}
Ted Kremenek02087932010-07-16 02:11:22 +0000994bool PrintfSpecifier::hasValidFieldWidth() const {
Tom Careb49ec692010-06-17 19:00:27 +0000995 if (FieldWidth.getHowSpecified() == OptionalAmount::NotSpecified)
996 return true;
997
998 // The field width is valid for all conversions except n
999 switch (CS.getKind()) {
Ted Kremenek516ef222010-07-20 20:04:10 +00001000 case ConversionSpecifier::nArg:
Tom Careb49ec692010-06-17 19:00:27 +00001001 return false;
1002
1003 default:
1004 return true;
1005 }
1006}