[PATCH] D45177: CStringChecker, check strlcpy/strlcat

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 27 16:57:56 PDT 2018


NoQ added inline comments.


================
Comment at: lib/StaticAnalyzer/Checkers/CStringChecker.cpp:106
+                        bool isAppending,
+                        bool canOverlap = false) const;
 
----------------
The fact that the regular `strcpy`/`strcat` isn't checked for overlaps looks like an omission to me. The difference between `strcpy`/`strcat` and `strlcpy`/`strlcat` is not in how it handles overlaps.


Repository:
  rC Clang

https://reviews.llvm.org/D45177





More information about the cfe-commits mailing list