diff options
author | Fariborz Jahanian <fjahanian@apple.com> | 2014-09-12 18:44:36 +0000 |
---|---|---|
committer | Fariborz Jahanian <fjahanian@apple.com> | 2014-09-12 18:44:36 +0000 |
commit | ab4fe98b4a0a8290efa07bd44d207750016c3c47 (patch) | |
tree | 6e86255a6841c56b340d0e24c64d3702401a20e2 /clang/lib/Sema/SemaChecking.cpp | |
parent | 7e954ea0630ec67d4e68cf18d540046256c5a465 (diff) | |
download | bcm5719-llvm-ab4fe98b4a0a8290efa07bd44d207750016c3c47.tar.gz bcm5719-llvm-ab4fe98b4a0a8290efa07bd44d207750016c3c47.zip |
patch to add missing warning on sizeof wrong parameter
for __builtin___strlcpy_chk/__builtin___strlcat_chk.
Patch by Jacques Fortier with monir change by me and
addition of test. rdar://18259539
llvm-svn: 217700
Diffstat (limited to 'clang/lib/Sema/SemaChecking.cpp')
-rw-r--r-- | clang/lib/Sema/SemaChecking.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/clang/lib/Sema/SemaChecking.cpp b/clang/lib/Sema/SemaChecking.cpp index ac5b68fb233..953b3f67d2f 100644 --- a/clang/lib/Sema/SemaChecking.cpp +++ b/clang/lib/Sema/SemaChecking.cpp @@ -4485,7 +4485,8 @@ void Sema::CheckStrlcpycatArguments(const CallExpr *Call, IdentifierInfo *FnName) { // Don't crash if the user has the wrong number of arguments - if (Call->getNumArgs() != 3) + unsigned NumArgs = Call->getNumArgs(); + if ((NumArgs != 3) && (NumArgs != 4)) return; const Expr *SrcArg = ignoreLiteralAdditions(Call->getArg(1), Context); |