fuzzilli: Fix stack-buffer-overflow in Fuzzilli driver on shutdown

The driver treated 0-byte reads (EOF) from the control pipe as errors
and attempted to log the `action_msg` buffer. This buffer was not
null-terminated, causing the logging operator to read past the stack
allocation and trigger an ASan `stack-buffer-overflow`.

Check for EOF explicitly and exit cleanly with a warning. Use
`base::as_string_view` to log received bytes safely based on the read
length. Distinguish between read errors (ERROR) and clean shutdown
(WARNING).

Bug: 477639524
Change-Id: Iad93d87acb5928f84da9f05f04698deba7693e57
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7513891
Commit-Queue: Giovanni Ortuno Urquidi <[email protected]>
Reviewed-by: Titouan Rigoudy <[email protected]>
Reviewed-by: Alex Kallam <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1575364}
NOKEYCHECK=True
GitOrigin-RevId: 14d045fe1842e926579368cbf51e7daad1eb5ea1
1 file changed
tree: 64a2d72e6f8bc8725e54600da8a27a9fbce0e3d1
  1. fuzzer_support_ios/
  2. fuzzers/
  3. fuzzilli/
  4. proto/
  5. renderer_fuzzing/
  6. research/
  7. tests/
  8. AFL_integration.md
  9. archive_corpus.py
  10. BUILD.gn
  11. confirm_fuzztest_init.cc
  12. confirm_fuzztests.py
  13. dictionary_generator.py
  14. efficient_fuzzing.md
  15. fuzzer_test.gni
  16. fuzzing_browsertests.md
  17. fuzztest_init_helper.cc
  18. fuzztest_init_helper.h
  19. fuzztest_wrapper.cpp
  20. gen_fuzzer_config.py
  21. gen_fuzzer_owners.py
  22. getting_started.md
  23. getting_started_with_libfuzzer.md
  24. libfuzzer_exports.h
  25. libFuzzer_integration.md
  26. libprotobuf-mutator.md
  27. OWNERS
  28. README.md
  29. reference.md
  30. reproducing.md
  31. unittest_main.cc
  32. zip_sources.py
README.md

Fuzz testing in Chromium

go/chrome-fuzzing

Fuzzing is a testing technique that feeds auto-generated inputs to a piece of target code in an attempt to crash the code. It's one of the most effective methods we have for finding security and stability issues (see go/fuzzing-success). You can learn more about the benefits of fuzzing at go/why-fuzz.

This documentation covers the in-process guided fuzzing approach employed by different fuzzing engines, such as libFuzzer or [AFL]. To learn more about out-of-process fuzzers, please refer to the Blackbox fuzzing page in the ClusterFuzz documentation.

Getting Started

In Chromium, you can easily create and submit fuzz targets. The targets are automatically discovered by buildbots, built with different fuzzing engines, then uploaded to the distributed ClusterFuzz fuzzing system to run at scale.

You should fuzz any code which absorbs inputs from untrusted sources, such as the web. If the code parses, decodes, or otherwise manipulates that input, it's an especially good idea to fuzz it.

Create your first fuzz target and submit it by stepping through our Getting Started Guide.

Advanced Topics

Further Reading

Trophies

Other Links