Skip to content

[libfuzzer] Clarify -max_len behavior on bigger files #123095

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jan 23, 2025

Conversation

chfast
Copy link
Member

@chfast chfast commented Jan 15, 2025

No description provided.

@llvmbot
Copy link
Member

llvmbot commented Jan 15, 2025

@llvm/pr-subscribers-compiler-rt-sanitizer

Author: Paweł Bylica (chfast)

Changes

Full diff: https://github.com/llvm/llvm-project/pull/123095.diff

1 Files Affected:

  • (modified) compiler-rt/lib/fuzzer/FuzzerFlags.def (+3-2)
diff --git a/compiler-rt/lib/fuzzer/FuzzerFlags.def b/compiler-rt/lib/fuzzer/FuzzerFlags.def
index fc3b3aa8c98ad5..b88458a497404c 100644
--- a/compiler-rt/lib/fuzzer/FuzzerFlags.def
+++ b/compiler-rt/lib/fuzzer/FuzzerFlags.def
@@ -14,8 +14,9 @@ FUZZER_FLAG_UNSIGNED(seed, 0, "Random seed. If 0, seed is generated.")
 FUZZER_FLAG_INT(runs, -1,
             "Number of individual test runs (-1 for infinite runs).")
 FUZZER_FLAG_INT(max_len, 0, "Maximum length of the test input. "
-    "If 0, libFuzzer tries to guess a good value based on the corpus "
-    "and reports it. ")
+  "Contents of corpus files are going to be truncated to this value. "
+  "If 0, libFuzzer tries to guess a good value based on the corpus "
+  "and reports it.")
 FUZZER_FLAG_INT(len_control, 100, "Try generating small inputs first, "
   "then try larger inputs over time.  Specifies the rate at which the length "
   "limit is increased (smaller == faster).  If 0, immediately try inputs with "

@chfast
Copy link
Member Author

chfast commented Jan 16, 2025

Will someone from @llvm/pr-subscribers-compiler-rt-sanitizer review this or should I look for a reviewer on my own?

@chfast chfast requested a review from vitalybuka January 17, 2025 12:56
@chfast chfast merged commit 091741a into llvm:main Jan 23, 2025
11 checks passed
@chfast chfast deleted the libfuzzer/clarify_max_len branch January 23, 2025 08:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants