[PATCH] D16161: [sancov] html report

Mike Aizatsky via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 14 13:19:28 PST 2016


aizatsky added a comment.

thanks. submitting.


================
Comment at: tools/sancov/sancov.cc:413
@@ +412,3 @@
+  for (char Ch : S) {
+    switch (Ch) {
+    case '&':
----------------
krasin wrote:
> 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.
> 
Yep. This is simply to put source code inside pre and not confuse the browser with < > & etc.


http://reviews.llvm.org/D16161





More information about the llvm-commits mailing list