[PATCH] D49722: [CStringSyntaxChecker] Check strlcat sizeof check

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added inline comments.


================
Comment at: lib/StaticAnalyzer/Checkers/CStringSyntaxChecker.cpp:275
+        os << "sizeof(" << DstName << ")";
+      else
+        os << "sizeof(<destination buffer>)";
----------------
Why can't this `else if` case be folded into the `strlcpy` case? There are lots of duplication.

`strlcpy` does not check `DstName.empty()` but this one does. Is there any cases I am missing?


https://reviews.llvm.org/D49722