[PATCH] D45177: CStringChecker, check strlcpy/strlcat

David CARLIER via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 4 22:29:57 PDT 2018


devnexen added inline comments.


================
Comment at: lib/StaticAnalyzer/Checkers/CStringChecker.cpp:1485
 
+  state = CheckOverlap(C, state, CE->getArg(2), Dst, srcExpr);
+
----------------
NoQ wrote:
> This crashes on the old tests for the checker. I guess that's because the normal `strcpy()` doesn't have three arguments (it counts from 0).
True I forgot those cases.


https://reviews.llvm.org/D45177





More information about the cfe-commits mailing list