diff options
| author | Ted Kremenek <kremenek@apple.com> | 2011-07-13 17:25:47 +0000 |
|---|---|---|
| committer | Ted Kremenek <kremenek@apple.com> | 2011-07-13 17:25:47 +0000 |
| commit | 1d3fb8af6811564846693f7f2d0b996553e21b4d (patch) | |
| tree | e366551ff0a6af206be36f29616c10e8033e74a2 /clang/lib/Analysis/FormatString.cpp | |
| parent | 88410049fa8c499e1c4bee3bde4b9da156f730d7 (diff) | |
| download | bcm5719-llvm-1d3fb8af6811564846693f7f2d0b996553e21b4d.tar.gz bcm5719-llvm-1d3fb8af6811564846693f7f2d0b996553e21b4d.zip | |
Fix inversion in argument type checking for format strings with conversion specifiers for character types.
llvm-svn: 135046
Diffstat (limited to 'clang/lib/Analysis/FormatString.cpp')
| -rw-r--r-- | clang/lib/Analysis/FormatString.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/clang/lib/Analysis/FormatString.cpp b/clang/lib/Analysis/FormatString.cpp index c1b5ea8a652..9a8639015c0 100644 --- a/clang/lib/Analysis/FormatString.cpp +++ b/clang/lib/Analysis/FormatString.cpp @@ -225,10 +225,10 @@ bool ArgTypeResult::matchesType(ASTContext &C, QualType argTy) const { break; case BuiltinType::Char_S: case BuiltinType::SChar: - return T == C.UnsignedCharTy; + return T == C.SignedCharTy; case BuiltinType::Char_U: case BuiltinType::UChar: - return T == C.SignedCharTy; + return T == C.UnsignedCharTy; case BuiltinType::Short: return T == C.UnsignedShortTy; case BuiltinType::UShort: |
