]> granicus.if.org Git - clang/commit
projects / clang / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 683a8b8) | patch
[analyzer] CStringSyntaxChecks: Fix an off-by-one error in the strlcat() check.
authorArtem Dergachev <artem.dergachev@gmail.com>
Fri, 8 Feb 2019 23:59:52 +0000 (23:59 +0000)
committerArtem Dergachev <artem.dergachev@gmail.com>
Fri, 8 Feb 2019 23:59:52 +0000 (23:59 +0000)
commit5bcf852d505e6e3d6d965f18c5fcd72ff5ef5a06
tree53d4b4ad353134ff1dd45294c8dc8af6b3e81360tree | snapshot
parent683a8b8183a89168d845870800e30c8ee733fb12commit | diff
[analyzer] CStringSyntaxChecks: Fix an off-by-one error in the strlcat() check.

oth strlcat and strlcpy cut off their safe bound for the argument value
at sizeof(destination). There's no need to subtract 1 in only one
of these cases.

Differential Revision: https://reviews.llvm.org/D57981

rdar://problem/47873212

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@353583 91177308-0d34-0410-b5e6-96231b3b80d8
lib/StaticAnalyzer/Checkers/CStringSyntaxChecker.cpp diff | blob | history
test/Analysis/cstring-syntax.c diff | blob | history
Unnamed repository; edit this file 'description' to name the repository.