diff options
| author | Sanjay Patel <spatel@rotateright.com> | 2021-07-31 15:13:42 -0400 |
|---|---|---|
| committer | Tom Stellard <tstellar@redhat.com> | 2021-08-02 22:58:39 -0700 |
| commit | d6974c010878cae1df5b27067230ee5dcbc63342 (patch) | |
| tree | bb1f8f9af6fd4a4e53eb0cec95763fca77d85f18 /llvm/lib/Analysis/TargetLibraryInfo.cpp | |
| parent | 60c388a4d66ea306a9da3fb7fea27227fcb6ab63 (diff) | |
| download | llvmorg-13.0.0-rc1.tar.gz | |
[Analysis] improve function signature checking for snprintfllvmorg-13.0.0-rc1
The check for size_t parameter 1 was already here for snprintf_chk,
but it wasn't applied to regular snprintf. This could lead to
mismatching and eventually crashing as shown in:
https://llvm.org/PR50885
(cherry picked from commit 7f5555776513f174729a686ed01270e23462aaf7)
Diffstat (limited to 'llvm/lib/Analysis/TargetLibraryInfo.cpp')
| -rw-r--r-- | llvm/lib/Analysis/TargetLibraryInfo.cpp | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/llvm/lib/Analysis/TargetLibraryInfo.cpp b/llvm/lib/Analysis/TargetLibraryInfo.cpp index 4a8818f2e2a8..c3a609ee4fe1 100644 --- a/llvm/lib/Analysis/TargetLibraryInfo.cpp +++ b/llvm/lib/Analysis/TargetLibraryInfo.cpp @@ -893,9 +893,10 @@ bool TargetLibraryInfoImpl::isValidProtoForLibFunc(const FunctionType &FTy, FTy.getReturnType()->isIntegerTy(32); case LibFunc_snprintf: - return (NumParams == 3 && FTy.getParamType(0)->isPointerTy() && - FTy.getParamType(2)->isPointerTy() && - FTy.getReturnType()->isIntegerTy(32)); + return NumParams == 3 && FTy.getParamType(0)->isPointerTy() && + IsSizeTTy(FTy.getParamType(1)) && + FTy.getParamType(2)->isPointerTy() && + FTy.getReturnType()->isIntegerTy(32); case LibFunc_snprintf_chk: return NumParams == 5 && FTy.getParamType(0)->isPointerTy() && |