tree 392c74cee4a6d4a002e2015f8fff66dc947fb4ff
parent aa8d73651a62991636d73e33d69ec18f53a9588c
author Xinhao Yuan <xinhaoyuan@google.com> 1791386951 -0700
committer Copybara-Service <copybara-worker@google.com> 1791386987 -0700

Let runner implement the engine ABI for the engine worker.

This consolidates the duplicated controller interaction in runner and engine worker.
There is some unique interaction left in the runner:

  - Dumping serailized FuzzTest config.

    Needed so that one can pass the fuzzing options to the test binary command when directly using the controller. But this uses C++-based struct. We should either drop this or make it dump controller options as strings, as part of the engine ABI.

  - Notifying the absense of custom mutator to use the controller's builtin mutator.

    Technically the runner has its own mutator for LLVMFuzzerMutate, but it's using the legacy ByteArrayMutator for minimal dependency, while the controller has the FuzzTest-based mutator. This should not be a problem once we remove the legacy support in runner (and let legacy fuzzers use the FuzzTest LLVM fuzzer wrapper instead)

Also, move fork server to the engine worker because we need to make sure that fork server starts before handling persistent mode.

PiperOrigin-RevId: 995125597