Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 1 | //= PrintfFormatStrings.cpp - Analysis of printf format strings --*- C++ -*-==// |
| 2 | // |
| 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 | |
| 15 | #include "clang/Analysis/Analyses/PrintfFormatString.h" |
Ted Kremenek | 33567d2 | 2010-01-29 22:59:32 +0000 | [diff] [blame] | 16 | #include "clang/AST/ASTContext.h" |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 17 | |
Ted Kremenek | 33567d2 | 2010-01-29 22:59:32 +0000 | [diff] [blame] | 18 | using clang::analyze_printf::ArgTypeResult; |
Ted Kremenek | efaff19 | 2010-02-27 01:41:03 +0000 | [diff] [blame] | 19 | using clang::analyze_printf::FormatSpecifier; |
Ted Kremenek | 74d56a1 | 2010-02-04 20:46:58 +0000 | [diff] [blame] | 20 | using clang::analyze_printf::FormatStringHandler; |
Ted Kremenek | efaff19 | 2010-02-27 01:41:03 +0000 | [diff] [blame] | 21 | using clang::analyze_printf::OptionalAmount; |
| 22 | using clang::analyze_printf::PositionContext; |
| 23 | |
Ted Kremenek | 808015a | 2010-01-29 03:16:21 +0000 | [diff] [blame] | 24 | using namespace clang; |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 25 | |
| 26 | namespace { |
| 27 | class FormatSpecifierResult { |
| 28 | FormatSpecifier FS; |
| 29 | const char *Start; |
Ted Kremenek | 26ac2e0 | 2010-01-29 02:40:24 +0000 | [diff] [blame] | 30 | bool Stop; |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 31 | public: |
Ted Kremenek | 26ac2e0 | 2010-01-29 02:40:24 +0000 | [diff] [blame] | 32 | FormatSpecifierResult(bool stop = false) |
| 33 | : Start(0), Stop(stop) {} |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 34 | FormatSpecifierResult(const char *start, |
Ted Kremenek | d2dcece | 2010-01-28 02:02:59 +0000 | [diff] [blame] | 35 | const FormatSpecifier &fs) |
Ted Kremenek | 26ac2e0 | 2010-01-29 02:40:24 +0000 | [diff] [blame] | 36 | : FS(fs), Start(start), Stop(false) {} |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 37 | |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 38 | |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 39 | const char *getStart() const { return Start; } |
Ted Kremenek | 26ac2e0 | 2010-01-29 02:40:24 +0000 | [diff] [blame] | 40 | bool shouldStop() const { return Stop; } |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 41 | bool hasValue() const { return Start != 0; } |
| 42 | const FormatSpecifier &getValue() const { |
| 43 | assert(hasValue()); |
| 44 | return FS; |
| 45 | } |
Ted Kremenek | d2dcece | 2010-01-28 02:02:59 +0000 | [diff] [blame] | 46 | const FormatSpecifier &getValue() { return FS; } |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 47 | }; |
| 48 | } // end anonymous namespace |
| 49 | |
| 50 | template <typename T> |
| 51 | class UpdateOnReturn { |
| 52 | T &ValueToUpdate; |
| 53 | const T &ValueToCopy; |
| 54 | public: |
| 55 | UpdateOnReturn(T &valueToUpdate, const T &valueToCopy) |
| 56 | : ValueToUpdate(valueToUpdate), ValueToCopy(valueToCopy) {} |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 57 | |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 58 | ~UpdateOnReturn() { |
| 59 | ValueToUpdate = ValueToCopy; |
| 60 | } |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 61 | }; |
| 62 | |
| 63 | //===----------------------------------------------------------------------===// |
| 64 | // Methods for parsing format strings. |
| 65 | //===----------------------------------------------------------------------===// |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 66 | |
Ted Kremenek | efaff19 | 2010-02-27 01:41:03 +0000 | [diff] [blame] | 67 | static OptionalAmount ParseAmount(const char *&Beg, const char *E) { |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 68 | const char *I = Beg; |
| 69 | UpdateOnReturn <const char*> UpdateBeg(Beg, I); |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 70 | |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 71 | unsigned accumulator = 0; |
Ted Kremenek | d49d877 | 2010-03-01 19:22:33 +0000 | [diff] [blame] | 72 | bool hasDigits = false; |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 73 | |
| 74 | for ( ; I != E; ++I) { |
| 75 | char c = *I; |
| 76 | if (c >= '0' && c <= '9') { |
Ted Kremenek | d49d877 | 2010-03-01 19:22:33 +0000 | [diff] [blame] | 77 | hasDigits = true; |
Ted Kremenek | fdb703a | 2010-03-25 03:59:09 +0000 | [diff] [blame^] | 78 | accumulator = (accumulator * 10) + (c - '0'); |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 79 | continue; |
| 80 | } |
| 81 | |
Ted Kremenek | d49d877 | 2010-03-01 19:22:33 +0000 | [diff] [blame] | 82 | if (hasDigits) |
Ted Kremenek | 7f70dc8 | 2010-02-26 19:18:41 +0000 | [diff] [blame] | 83 | return OptionalAmount(OptionalAmount::Constant, accumulator, Beg); |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 84 | |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 85 | break; |
| 86 | } |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 87 | |
| 88 | return OptionalAmount(); |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 89 | } |
| 90 | |
Ted Kremenek | efaff19 | 2010-02-27 01:41:03 +0000 | [diff] [blame] | 91 | static OptionalAmount ParseNonPositionAmount(const char *&Beg, const char *E, |
| 92 | unsigned &argIndex) { |
| 93 | if (*Beg == '*') { |
| 94 | ++Beg; |
| 95 | return OptionalAmount(OptionalAmount::Arg, argIndex++, Beg); |
| 96 | } |
| 97 | |
| 98 | return ParseAmount(Beg, E); |
| 99 | } |
| 100 | |
| 101 | static OptionalAmount ParsePositionAmount(FormatStringHandler &H, |
| 102 | const char *Start, |
| 103 | const char *&Beg, const char *E, |
| 104 | PositionContext p) { |
| 105 | if (*Beg == '*') { |
| 106 | const char *I = Beg + 1; |
| 107 | const OptionalAmount &Amt = ParseAmount(I, E); |
| 108 | |
| 109 | if (Amt.getHowSpecified() == OptionalAmount::NotSpecified) { |
| 110 | H.HandleInvalidPosition(Beg, I - Beg, p); |
| 111 | return OptionalAmount(false); |
| 112 | } |
| 113 | |
| 114 | if (I== E) { |
| 115 | // No more characters left? |
| 116 | H.HandleIncompleteFormatSpecifier(Start, E - Start); |
| 117 | return OptionalAmount(false); |
| 118 | } |
| 119 | |
Ted Kremenek | d49d877 | 2010-03-01 19:22:33 +0000 | [diff] [blame] | 120 | assert(Amt.getHowSpecified() == OptionalAmount::Constant); |
| 121 | |
Ted Kremenek | efaff19 | 2010-02-27 01:41:03 +0000 | [diff] [blame] | 122 | if (*I == '$') { |
Ted Kremenek | d49d877 | 2010-03-01 19:22:33 +0000 | [diff] [blame] | 123 | // Special case: '*0$', since this is an easy mistake. |
| 124 | if (Amt.getConstantAmount() == 0) { |
| 125 | H.HandleZeroPosition(Beg, I - Beg + 1); |
| 126 | return OptionalAmount(false); |
| 127 | } |
| 128 | |
Ted Kremenek | efaff19 | 2010-02-27 01:41:03 +0000 | [diff] [blame] | 129 | const char *Tmp = Beg; |
| 130 | Beg = ++I; |
Ted Kremenek | d49d877 | 2010-03-01 19:22:33 +0000 | [diff] [blame] | 131 | |
Ted Kremenek | efaff19 | 2010-02-27 01:41:03 +0000 | [diff] [blame] | 132 | return OptionalAmount(OptionalAmount::Arg, Amt.getConstantAmount() - 1, |
| 133 | Tmp); |
| 134 | } |
| 135 | |
| 136 | H.HandleInvalidPosition(Beg, I - Beg, p); |
| 137 | return OptionalAmount(false); |
| 138 | } |
| 139 | |
| 140 | return ParseAmount(Beg, E); |
| 141 | } |
| 142 | |
| 143 | static bool ParsePrecision(FormatStringHandler &H, FormatSpecifier &FS, |
| 144 | const char *Start, const char *&Beg, const char *E, |
| 145 | unsigned *argIndex) { |
| 146 | if (argIndex) { |
| 147 | FS.setPrecision(ParseNonPositionAmount(Beg, E, *argIndex)); |
| 148 | } |
| 149 | else { |
| 150 | const OptionalAmount Amt = ParsePositionAmount(H, Start, Beg, E, |
| 151 | analyze_printf::PrecisionPos); |
| 152 | if (Amt.isInvalid()) |
| 153 | return true; |
| 154 | FS.setPrecision(Amt); |
| 155 | } |
| 156 | return false; |
| 157 | } |
| 158 | |
| 159 | static bool ParseFieldWidth(FormatStringHandler &H, FormatSpecifier &FS, |
| 160 | const char *Start, const char *&Beg, const char *E, |
| 161 | unsigned *argIndex) { |
| 162 | // FIXME: Support negative field widths. |
| 163 | if (argIndex) { |
| 164 | FS.setFieldWidth(ParseNonPositionAmount(Beg, E, *argIndex)); |
| 165 | } |
| 166 | else { |
| 167 | const OptionalAmount Amt = ParsePositionAmount(H, Start, Beg, E, |
| 168 | analyze_printf::FieldWidthPos); |
| 169 | if (Amt.isInvalid()) |
| 170 | return true; |
| 171 | FS.setFieldWidth(Amt); |
| 172 | } |
| 173 | return false; |
| 174 | } |
| 175 | |
| 176 | |
| 177 | static bool ParseArgPosition(FormatStringHandler &H, |
| 178 | FormatSpecifier &FS, const char *Start, |
| 179 | const char *&Beg, const char *E) { |
| 180 | |
| 181 | using namespace clang::analyze_printf; |
| 182 | const char *I = Beg; |
| 183 | |
| 184 | const OptionalAmount &Amt = ParseAmount(I, E); |
| 185 | |
| 186 | if (I == E) { |
| 187 | // No more characters left? |
| 188 | H.HandleIncompleteFormatSpecifier(Start, E - Start); |
| 189 | return true; |
| 190 | } |
| 191 | |
| 192 | if (Amt.getHowSpecified() == OptionalAmount::Constant && *(I++) == '$') { |
Ted Kremenek | d49d877 | 2010-03-01 19:22:33 +0000 | [diff] [blame] | 193 | // Special case: '%0$', since this is an easy mistake. |
| 194 | if (Amt.getConstantAmount() == 0) { |
| 195 | H.HandleZeroPosition(Start, I - Start); |
| 196 | return true; |
| 197 | } |
| 198 | |
Ted Kremenek | efaff19 | 2010-02-27 01:41:03 +0000 | [diff] [blame] | 199 | FS.setArgIndex(Amt.getConstantAmount() - 1); |
| 200 | FS.setUsesPositionalArg(); |
| 201 | // Update the caller's pointer if we decided to consume |
| 202 | // these characters. |
| 203 | Beg = I; |
| 204 | return false; |
| 205 | } |
| 206 | |
Ted Kremenek | efaff19 | 2010-02-27 01:41:03 +0000 | [diff] [blame] | 207 | return false; |
| 208 | } |
| 209 | |
Ted Kremenek | 808015a | 2010-01-29 03:16:21 +0000 | [diff] [blame] | 210 | static FormatSpecifierResult ParseFormatSpecifier(FormatStringHandler &H, |
Ted Kremenek | 74d56a1 | 2010-02-04 20:46:58 +0000 | [diff] [blame] | 211 | const char *&Beg, |
Ted Kremenek | 7f70dc8 | 2010-02-26 19:18:41 +0000 | [diff] [blame] | 212 | const char *E, |
| 213 | unsigned &argIndex) { |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 214 | |
Ted Kremenek | 4b220fa | 2010-01-29 02:13:53 +0000 | [diff] [blame] | 215 | using namespace clang::analyze_printf; |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 216 | |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 217 | const char *I = Beg; |
Ted Kremenek | c7ae51a | 2010-01-28 00:02:05 +0000 | [diff] [blame] | 218 | const char *Start = 0; |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 219 | UpdateOnReturn <const char*> UpdateBeg(Beg, I); |
| 220 | |
| 221 | // Look for a '%' character that indicates the start of a format specifier. |
Ted Kremenek | e729acb | 2010-01-28 23:56:52 +0000 | [diff] [blame] | 222 | for ( ; I != E ; ++I) { |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 223 | char c = *I; |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 224 | if (c == '\0') { |
| 225 | // Detect spurious null characters, which are likely errors. |
| 226 | H.HandleNullChar(I); |
| 227 | return true; |
| 228 | } |
| 229 | if (c == '%') { |
Ted Kremenek | e729acb | 2010-01-28 23:56:52 +0000 | [diff] [blame] | 230 | Start = I++; // Record the start of the format specifier. |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 231 | break; |
| 232 | } |
| 233 | } |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 234 | |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 235 | // No format specifier found? |
| 236 | if (!Start) |
| 237 | return false; |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 238 | |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 239 | if (I == E) { |
| 240 | // No more characters left? |
Ted Kremenek | 808015a | 2010-01-29 03:16:21 +0000 | [diff] [blame] | 241 | H.HandleIncompleteFormatSpecifier(Start, E - Start); |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 242 | return true; |
| 243 | } |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 244 | |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 245 | FormatSpecifier FS; |
Ted Kremenek | efaff19 | 2010-02-27 01:41:03 +0000 | [diff] [blame] | 246 | if (ParseArgPosition(H, FS, Start, I, E)) |
| 247 | return true; |
| 248 | |
| 249 | if (I == E) { |
| 250 | // No more characters left? |
| 251 | H.HandleIncompleteFormatSpecifier(Start, E - Start); |
| 252 | return true; |
| 253 | } |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 254 | |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 255 | // Look for flags (if any). |
| 256 | bool hasMore = true; |
| 257 | for ( ; I != E; ++I) { |
| 258 | switch (*I) { |
| 259 | default: hasMore = false; break; |
| 260 | case '-': FS.setIsLeftJustified(); break; |
| 261 | case '+': FS.setHasPlusPrefix(); break; |
| 262 | case ' ': FS.setHasSpacePrefix(); break; |
| 263 | case '#': FS.setHasAlternativeForm(); break; |
| 264 | case '0': FS.setHasLeadingZeros(); break; |
| 265 | } |
| 266 | if (!hasMore) |
| 267 | break; |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 268 | } |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 269 | |
| 270 | if (I == E) { |
| 271 | // No more characters left? |
Ted Kremenek | 808015a | 2010-01-29 03:16:21 +0000 | [diff] [blame] | 272 | H.HandleIncompleteFormatSpecifier(Start, E - Start); |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 273 | return true; |
| 274 | } |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 275 | |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 276 | // Look for the field width (if any). |
Ted Kremenek | efaff19 | 2010-02-27 01:41:03 +0000 | [diff] [blame] | 277 | if (ParseFieldWidth(H, FS, Start, I, E, |
| 278 | FS.usesPositionalArg() ? 0 : &argIndex)) |
| 279 | return true; |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 280 | |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 281 | if (I == E) { |
| 282 | // No more characters left? |
Ted Kremenek | 808015a | 2010-01-29 03:16:21 +0000 | [diff] [blame] | 283 | H.HandleIncompleteFormatSpecifier(Start, E - Start); |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 284 | return true; |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 285 | } |
| 286 | |
| 287 | // Look for the precision (if any). |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 288 | if (*I == '.') { |
Ted Kremenek | 808015a | 2010-01-29 03:16:21 +0000 | [diff] [blame] | 289 | ++I; |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 290 | if (I == E) { |
Ted Kremenek | 808015a | 2010-01-29 03:16:21 +0000 | [diff] [blame] | 291 | H.HandleIncompleteFormatSpecifier(Start, E - Start); |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 292 | return true; |
| 293 | } |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 294 | |
Ted Kremenek | efaff19 | 2010-02-27 01:41:03 +0000 | [diff] [blame] | 295 | if (ParsePrecision(H, FS, Start, I, E, |
| 296 | FS.usesPositionalArg() ? 0 : &argIndex)) |
| 297 | return true; |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 298 | |
| 299 | if (I == E) { |
| 300 | // No more characters left? |
Ted Kremenek | 808015a | 2010-01-29 03:16:21 +0000 | [diff] [blame] | 301 | H.HandleIncompleteFormatSpecifier(Start, E - Start); |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 302 | return true; |
| 303 | } |
| 304 | } |
| 305 | |
| 306 | // Look for the length modifier. |
| 307 | LengthModifier lm = None; |
| 308 | switch (*I) { |
| 309 | default: |
| 310 | break; |
| 311 | case 'h': |
| 312 | ++I; |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 313 | lm = (I != E && *I == 'h') ? ++I, AsChar : AsShort; |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 314 | break; |
| 315 | case 'l': |
| 316 | ++I; |
| 317 | lm = (I != E && *I == 'l') ? ++I, AsLongLong : AsLong; |
| 318 | break; |
| 319 | case 'j': lm = AsIntMax; ++I; break; |
| 320 | case 'z': lm = AsSizeT; ++I; break; |
| 321 | case 't': lm = AsPtrDiff; ++I; break; |
| 322 | case 'L': lm = AsLongDouble; ++I; break; |
Daniel Dunbar | 01aefc6 | 2010-01-30 15:49:20 +0000 | [diff] [blame] | 323 | case 'q': lm = AsLongLong; ++I; break; |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 324 | } |
| 325 | FS.setLengthModifier(lm); |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 326 | |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 327 | if (I == E) { |
| 328 | // No more characters left? |
Ted Kremenek | 808015a | 2010-01-29 03:16:21 +0000 | [diff] [blame] | 329 | H.HandleIncompleteFormatSpecifier(Start, E - Start); |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 330 | return true; |
| 331 | } |
Ted Kremenek | df17f9d | 2010-02-09 00:04:09 +0000 | [diff] [blame] | 332 | |
Ted Kremenek | 4dcb18f | 2010-01-29 20:29:53 +0000 | [diff] [blame] | 333 | if (*I == '\0') { |
Ted Kremenek | df17f9d | 2010-02-09 00:04:09 +0000 | [diff] [blame] | 334 | // Detect spurious null characters, which are likely errors. |
| 335 | H.HandleNullChar(I); |
| 336 | return true; |
Ted Kremenek | 4dcb18f | 2010-01-29 20:29:53 +0000 | [diff] [blame] | 337 | } |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 338 | |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 339 | // Finally, look for the conversion specifier. |
Ted Kremenek | a8d8fec | 2010-01-28 02:46:17 +0000 | [diff] [blame] | 340 | const char *conversionPosition = I++; |
Ted Kremenek | 26ac2e0 | 2010-01-29 02:40:24 +0000 | [diff] [blame] | 341 | ConversionSpecifier::Kind k = ConversionSpecifier::InvalidSpecifier; |
Ted Kremenek | a8d8fec | 2010-01-28 02:46:17 +0000 | [diff] [blame] | 342 | switch (*conversionPosition) { |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 343 | default: |
Ted Kremenek | 26ac2e0 | 2010-01-29 02:40:24 +0000 | [diff] [blame] | 344 | break; |
Ted Kremenek | c7cbb9b | 2010-01-28 00:55:28 +0000 | [diff] [blame] | 345 | // C99: 7.19.6.1 (section 8). |
Ted Kremenek | 87260c7 | 2010-02-24 00:05:54 +0000 | [diff] [blame] | 346 | case '%': k = ConversionSpecifier::PercentArg; break; |
| 347 | case 'A': k = ConversionSpecifier::AArg; break; |
| 348 | case 'E': k = ConversionSpecifier::EArg; break; |
| 349 | case 'F': k = ConversionSpecifier::FArg; break; |
| 350 | case 'G': k = ConversionSpecifier::GArg; break; |
| 351 | case 'X': k = ConversionSpecifier::XArg; break; |
| 352 | case 'a': k = ConversionSpecifier::aArg; break; |
| 353 | case 'c': k = ConversionSpecifier::IntAsCharArg; break; |
Ted Kremenek | a8d8fec | 2010-01-28 02:46:17 +0000 | [diff] [blame] | 354 | case 'd': k = ConversionSpecifier::dArg; break; |
Ted Kremenek | 87260c7 | 2010-02-24 00:05:54 +0000 | [diff] [blame] | 355 | case 'e': k = ConversionSpecifier::eArg; break; |
| 356 | case 'f': k = ConversionSpecifier::fArg; break; |
| 357 | case 'g': k = ConversionSpecifier::gArg; break; |
Ted Kremenek | a8d8fec | 2010-01-28 02:46:17 +0000 | [diff] [blame] | 358 | case 'i': k = ConversionSpecifier::iArg; break; |
Ted Kremenek | 87260c7 | 2010-02-24 00:05:54 +0000 | [diff] [blame] | 359 | case 'n': k = ConversionSpecifier::OutIntPtrArg; break; |
Ted Kremenek | a8d8fec | 2010-01-28 02:46:17 +0000 | [diff] [blame] | 360 | case 'o': k = ConversionSpecifier::oArg; break; |
Ted Kremenek | 87260c7 | 2010-02-24 00:05:54 +0000 | [diff] [blame] | 361 | case 'p': k = ConversionSpecifier::VoidPtrArg; break; |
| 362 | case 's': k = ConversionSpecifier::CStrArg; break; |
Ted Kremenek | a8d8fec | 2010-01-28 02:46:17 +0000 | [diff] [blame] | 363 | case 'u': k = ConversionSpecifier::uArg; break; |
| 364 | case 'x': k = ConversionSpecifier::xArg; break; |
Ted Kremenek | 87260c7 | 2010-02-24 00:05:54 +0000 | [diff] [blame] | 365 | // Mac OS X (unicode) specific |
| 366 | case 'C': k = ConversionSpecifier::CArg; break; |
| 367 | case 'S': k = ConversionSpecifier::UnicodeStrArg; break; |
Ted Kremenek | c7cbb9b | 2010-01-28 00:55:28 +0000 | [diff] [blame] | 368 | // Objective-C. |
Ted Kremenek | 4dcb18f | 2010-01-29 20:29:53 +0000 | [diff] [blame] | 369 | case '@': k = ConversionSpecifier::ObjCObjArg; break; |
Ted Kremenek | df17f9d | 2010-02-09 00:04:09 +0000 | [diff] [blame] | 370 | // Glibc specific. |
Ted Kremenek | 4dcb18f | 2010-01-29 20:29:53 +0000 | [diff] [blame] | 371 | case 'm': k = ConversionSpecifier::PrintErrno; break; |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 372 | } |
Ted Kremenek | 7f70dc8 | 2010-02-26 19:18:41 +0000 | [diff] [blame] | 373 | ConversionSpecifier CS(conversionPosition, k); |
| 374 | FS.setConversionSpecifier(CS); |
Ted Kremenek | efaff19 | 2010-02-27 01:41:03 +0000 | [diff] [blame] | 375 | if (CS.consumesDataArgument() && !FS.usesPositionalArg()) |
Ted Kremenek | 7f70dc8 | 2010-02-26 19:18:41 +0000 | [diff] [blame] | 376 | FS.setArgIndex(argIndex++); |
Ted Kremenek | 26ac2e0 | 2010-01-29 02:40:24 +0000 | [diff] [blame] | 377 | |
| 378 | if (k == ConversionSpecifier::InvalidSpecifier) { |
Ted Kremenek | 7f70dc8 | 2010-02-26 19:18:41 +0000 | [diff] [blame] | 379 | // Assume the conversion takes one argument. |
| 380 | return !H.HandleInvalidConversionSpecifier(FS, Beg, I - Beg); |
Ted Kremenek | 26ac2e0 | 2010-01-29 02:40:24 +0000 | [diff] [blame] | 381 | } |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 382 | return FormatSpecifierResult(Start, FS); |
| 383 | } |
| 384 | |
Ted Kremenek | 74d56a1 | 2010-02-04 20:46:58 +0000 | [diff] [blame] | 385 | bool clang::analyze_printf::ParseFormatString(FormatStringHandler &H, |
Ted Kremenek | a8d8fec | 2010-01-28 02:46:17 +0000 | [diff] [blame] | 386 | const char *I, const char *E) { |
Ted Kremenek | 7f70dc8 | 2010-02-26 19:18:41 +0000 | [diff] [blame] | 387 | |
| 388 | unsigned argIndex = 0; |
| 389 | |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 390 | // Keep looking for a format specifier until we have exhausted the string. |
| 391 | while (I != E) { |
Ted Kremenek | 7f70dc8 | 2010-02-26 19:18:41 +0000 | [diff] [blame] | 392 | const FormatSpecifierResult &FSR = ParseFormatSpecifier(H, I, E, argIndex); |
Ted Kremenek | 26ac2e0 | 2010-01-29 02:40:24 +0000 | [diff] [blame] | 393 | // Did a fail-stop error of any kind occur when parsing the specifier? |
| 394 | // If so, don't do any more processing. |
| 395 | if (FSR.shouldStop()) |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 396 | return true;; |
Ted Kremenek | 26ac2e0 | 2010-01-29 02:40:24 +0000 | [diff] [blame] | 397 | // Did we exhaust the string or encounter an error that |
| 398 | // we can recover from? |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 399 | if (!FSR.hasValue()) |
Ted Kremenek | 26ac2e0 | 2010-01-29 02:40:24 +0000 | [diff] [blame] | 400 | continue; |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 401 | // We have a format specifier. Pass it to the callback. |
Ted Kremenek | a8d8fec | 2010-01-28 02:46:17 +0000 | [diff] [blame] | 402 | if (!H.HandleFormatSpecifier(FSR.getValue(), FSR.getStart(), |
| 403 | I - FSR.getStart())) |
Ted Kremenek | 4dcb18f | 2010-01-29 20:29:53 +0000 | [diff] [blame] | 404 | return true; |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 405 | } |
| 406 | assert(I == E && "Format string not exhausted"); |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 407 | return false; |
| 408 | } |
Ted Kremenek | 8f0a1c7 | 2010-01-27 23:43:25 +0000 | [diff] [blame] | 409 | |
| 410 | FormatStringHandler::~FormatStringHandler() {} |
Ted Kremenek | 33567d2 | 2010-01-29 22:59:32 +0000 | [diff] [blame] | 411 | |
| 412 | //===----------------------------------------------------------------------===// |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 413 | // Methods on ArgTypeResult. |
| 414 | //===----------------------------------------------------------------------===// |
| 415 | |
| 416 | bool ArgTypeResult::matchesType(ASTContext &C, QualType argTy) const { |
| 417 | assert(isValid()); |
| 418 | |
| 419 | if (K == UnknownTy) |
| 420 | return true; |
| 421 | |
| 422 | if (K == SpecificTy) { |
| 423 | argTy = C.getCanonicalType(argTy).getUnqualifiedType(); |
| 424 | |
| 425 | if (T == argTy) |
| 426 | return true; |
| 427 | |
| 428 | if (const BuiltinType *BT = argTy->getAs<BuiltinType>()) |
| 429 | switch (BT->getKind()) { |
| 430 | default: |
| 431 | break; |
| 432 | case BuiltinType::Char_S: |
| 433 | case BuiltinType::SChar: |
| 434 | return T == C.UnsignedCharTy; |
| 435 | case BuiltinType::Char_U: |
| 436 | case BuiltinType::UChar: |
| 437 | return T == C.SignedCharTy; |
| 438 | case BuiltinType::Short: |
| 439 | return T == C.UnsignedShortTy; |
| 440 | case BuiltinType::UShort: |
| 441 | return T == C.ShortTy; |
| 442 | case BuiltinType::Int: |
| 443 | return T == C.UnsignedIntTy; |
| 444 | case BuiltinType::UInt: |
| 445 | return T == C.IntTy; |
| 446 | case BuiltinType::Long: |
| 447 | return T == C.UnsignedLongTy; |
| 448 | case BuiltinType::ULong: |
| 449 | return T == C.LongTy; |
| 450 | case BuiltinType::LongLong: |
| 451 | return T == C.UnsignedLongLongTy; |
| 452 | case BuiltinType::ULongLong: |
| 453 | return T == C.LongLongTy; |
| 454 | } |
| 455 | |
| 456 | return false; |
| 457 | } |
| 458 | |
| 459 | if (K == CStrTy) { |
| 460 | const PointerType *PT = argTy->getAs<PointerType>(); |
| 461 | if (!PT) |
| 462 | return false; |
| 463 | |
| 464 | QualType pointeeTy = PT->getPointeeType(); |
| 465 | |
| 466 | if (const BuiltinType *BT = pointeeTy->getAs<BuiltinType>()) |
| 467 | switch (BT->getKind()) { |
| 468 | case BuiltinType::Void: |
| 469 | case BuiltinType::Char_U: |
| 470 | case BuiltinType::UChar: |
| 471 | case BuiltinType::Char_S: |
| 472 | case BuiltinType::SChar: |
| 473 | return true; |
| 474 | default: |
| 475 | break; |
| 476 | } |
| 477 | |
| 478 | return false; |
| 479 | } |
| 480 | |
| 481 | if (K == WCStrTy) { |
| 482 | const PointerType *PT = argTy->getAs<PointerType>(); |
| 483 | if (!PT) |
| 484 | return false; |
| 485 | |
Ted Kremenek | 87260c7 | 2010-02-24 00:05:54 +0000 | [diff] [blame] | 486 | QualType pointeeTy = |
| 487 | C.getCanonicalType(PT->getPointeeType()).getUnqualifiedType(); |
| 488 | |
| 489 | return pointeeTy == C.getWCharType(); |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 490 | } |
| 491 | |
| 492 | return false; |
| 493 | } |
| 494 | |
| 495 | QualType ArgTypeResult::getRepresentativeType(ASTContext &C) const { |
| 496 | assert(isValid()); |
| 497 | if (K == SpecificTy) |
| 498 | return T; |
| 499 | if (K == CStrTy) |
| 500 | return C.getPointerType(C.CharTy); |
| 501 | if (K == WCStrTy) |
Ted Kremenek | 87260c7 | 2010-02-24 00:05:54 +0000 | [diff] [blame] | 502 | return C.getPointerType(C.getWCharType()); |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 503 | if (K == ObjCPointerTy) |
| 504 | return C.ObjCBuiltinIdTy; |
| 505 | |
| 506 | return QualType(); |
| 507 | } |
| 508 | |
| 509 | //===----------------------------------------------------------------------===// |
| 510 | // Methods on OptionalAmount. |
| 511 | //===----------------------------------------------------------------------===// |
| 512 | |
| 513 | ArgTypeResult OptionalAmount::getArgType(ASTContext &Ctx) const { |
| 514 | return Ctx.IntTy; |
| 515 | } |
| 516 | |
| 517 | //===----------------------------------------------------------------------===// |
Ted Kremenek | 33567d2 | 2010-01-29 22:59:32 +0000 | [diff] [blame] | 518 | // Methods on FormatSpecifier. |
| 519 | //===----------------------------------------------------------------------===// |
| 520 | |
| 521 | ArgTypeResult FormatSpecifier::getArgType(ASTContext &Ctx) const { |
| 522 | if (!CS.consumesDataArgument()) |
| 523 | return ArgTypeResult::Invalid(); |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 524 | |
Ted Kremenek | 33567d2 | 2010-01-29 22:59:32 +0000 | [diff] [blame] | 525 | if (CS.isIntArg()) |
| 526 | switch (LM) { |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 527 | case AsLongDouble: |
Ted Kremenek | 33567d2 | 2010-01-29 22:59:32 +0000 | [diff] [blame] | 528 | return ArgTypeResult::Invalid(); |
| 529 | case None: return Ctx.IntTy; |
| 530 | case AsChar: return Ctx.SignedCharTy; |
| 531 | case AsShort: return Ctx.ShortTy; |
| 532 | case AsLong: return Ctx.LongTy; |
| 533 | case AsLongLong: return Ctx.LongLongTy; |
| 534 | case AsIntMax: |
| 535 | // FIXME: Return unknown for now. |
| 536 | return ArgTypeResult(); |
| 537 | case AsSizeT: return Ctx.getSizeType(); |
| 538 | case AsPtrDiff: return Ctx.getPointerDiffType(); |
| 539 | } |
| 540 | |
| 541 | if (CS.isUIntArg()) |
| 542 | switch (LM) { |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 543 | case AsLongDouble: |
Ted Kremenek | 33567d2 | 2010-01-29 22:59:32 +0000 | [diff] [blame] | 544 | return ArgTypeResult::Invalid(); |
| 545 | case None: return Ctx.UnsignedIntTy; |
| 546 | case AsChar: return Ctx.UnsignedCharTy; |
| 547 | case AsShort: return Ctx.UnsignedShortTy; |
| 548 | case AsLong: return Ctx.UnsignedLongTy; |
| 549 | case AsLongLong: return Ctx.UnsignedLongLongTy; |
| 550 | case AsIntMax: |
| 551 | // FIXME: Return unknown for now. |
| 552 | return ArgTypeResult(); |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 553 | case AsSizeT: |
Ted Kremenek | 33567d2 | 2010-01-29 22:59:32 +0000 | [diff] [blame] | 554 | // FIXME: How to get the corresponding unsigned |
| 555 | // version of size_t? |
| 556 | return ArgTypeResult(); |
| 557 | case AsPtrDiff: |
| 558 | // FIXME: How to get the corresponding unsigned |
| 559 | // version of ptrdiff_t? |
| 560 | return ArgTypeResult(); |
| 561 | } |
Ted Kremenek | 4e4b30e | 2010-02-16 01:46:59 +0000 | [diff] [blame] | 562 | |
Ted Kremenek | f911eba | 2010-02-01 23:23:50 +0000 | [diff] [blame] | 563 | if (CS.isDoubleArg()) { |
| 564 | if (LM == AsLongDouble) |
| 565 | return Ctx.LongDoubleTy; |
Ted Kremenek | c9a89fe | 2010-01-30 01:02:18 +0000 | [diff] [blame] | 566 | return Ctx.DoubleTy; |
Ted Kremenek | f911eba | 2010-02-01 23:23:50 +0000 | [diff] [blame] | 567 | } |
Ted Kremenek | 7f70dc8 | 2010-02-26 19:18:41 +0000 | [diff] [blame] | 568 | |
Ted Kremenek | 87260c7 | 2010-02-24 00:05:54 +0000 | [diff] [blame] | 569 | switch (CS.getKind()) { |
| 570 | case ConversionSpecifier::CStrArg: |
| 571 | return ArgTypeResult(LM == AsWideChar ? ArgTypeResult::WCStrTy : ArgTypeResult::CStrTy); |
| 572 | case ConversionSpecifier::UnicodeStrArg: |
| 573 | // FIXME: This appears to be Mac OS X specific. |
| 574 | return ArgTypeResult::WCStrTy; |
| 575 | case ConversionSpecifier::CArg: |
Ted Kremenek | 7f70dc8 | 2010-02-26 19:18:41 +0000 | [diff] [blame] | 576 | return Ctx.WCharTy; |
Ted Kremenek | 87260c7 | 2010-02-24 00:05:54 +0000 | [diff] [blame] | 577 | default: |
| 578 | break; |
| 579 | } |
Ted Kremenek | 7f70dc8 | 2010-02-26 19:18:41 +0000 | [diff] [blame] | 580 | |
Ted Kremenek | 33567d2 | 2010-01-29 22:59:32 +0000 | [diff] [blame] | 581 | // FIXME: Handle other cases. |
Ted Kremenek | 40888ad | 2010-01-29 23:00:35 +0000 | [diff] [blame] | 582 | return ArgTypeResult(); |
Ted Kremenek | 33567d2 | 2010-01-29 22:59:32 +0000 | [diff] [blame] | 583 | } |
| 584 | |