[PATCH] D16161: [sancov] html report

Ivan Krasin via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 13 16:20:30 PST 2016


krasin added a comment.

LGTM


================
Comment at: tools/sancov/sancov.cc:413
@@ +412,3 @@
+  for (char Ch : S) {
+    switch (Ch) {
+    case '&':
----------------
I assume that we don't have security implications here, as the input is not under a stranger control, and this escaping is just to make stuff working.



http://reviews.llvm.org/D16161





More information about the llvm-commits mailing list