commit ced2232227f1df567cd6b324aa0ed87840f99615
author Max Moroz <mmoroz@chromium.org> Tue Feb 18 09:57:16 2020 -0800
committer Copybara-Service <copybara-worker@google.com> Thu Mar 25 23:20:46 2021 -0700
tree 4c48cb5d538f8002f915e67c5ce8c1d32c6b83d1
parent c5b7025ab419ff8b32d7638a72879b1a4f97970d

[libFuzzer] Fix denominator in the "inputs have DFT" log line.

Summary:
The number of "inputs have the Data Flow Trace" cannot be greater than
the number of inputs touching the focus function. The existing message is rather
confusing as the same log would mention a greater total number of traces a few
lines above.

Reviewers: kcc, metzman

Subscribers: #sanitizers, llvm-commits

Tags: #sanitizers, #llvm

Differential Revision: https://reviews.llvm.org/D74779

GitOrigin-RevId: 076475713c236081a3247a53e9dbab9043c3eac2

FuzzerLoop.cpp

diff --git a/FuzzerLoop.cpp b/FuzzerLoop.cpp
index 09f319d..273c629 100644
--- a/FuzzerLoop.cpp
+++ b/FuzzerLoop.cpp
@@ -770,12 +770,14 @@
   }
 
   PrintStats("INITED");
-  if (!Options.FocusFunction.empty())
+  if (!Options.FocusFunction.empty()) {
     Printf("INFO: %zd/%zd inputs touch the focus function\n",
            Corpus.NumInputsThatTouchFocusFunction(), Corpus.size());
-  if (!Options.DataFlowTrace.empty())
-    Printf("INFO: %zd/%zd inputs have the Data Flow Trace\n",
-           Corpus.NumInputsWithDataFlowTrace(), Corpus.size());
+    if (!Options.DataFlowTrace.empty())
+      Printf("INFO: %zd/%zd inputs have the Data Flow Trace\n",
+             Corpus.NumInputsWithDataFlowTrace(),
+             Corpus.NumInputsThatTouchFocusFunction());
+  }
 
   if (Corpus.empty() && Options.MaxNumberOfRuns) {
     Printf("ERROR: no interesting inputs were found. "