Fix the inherited Fuzzer class for the compatibility mode.

PiperOrigin-RevId: 718374858
google · Jan 22, 2025 · edd030b · edd030b
1 parent 130dd97
commit edd030b
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/fuzztest/internal/compatibility_mode.cc b/fuzztest/internal/compatibility_mode.cc
@@ -65,12 +65,12 @@ FuzzTestExternalEngineAdaptor::FuzzTestExternalEngineAdaptor(
     const FuzzTest& test, std::unique_ptr<Driver> fixture_driver)
     : test_(test), fixture_driver_staging_(std::move(fixture_driver)) {}
 
-void FuzzTestExternalEngineAdaptor::RunInUnitTestMode(
+bool FuzzTestExternalEngineAdaptor::RunInUnitTestMode(
     const Configuration& configuration) {
-  GetFuzzerImpl().RunInUnitTestMode(configuration);
+  return GetFuzzerImpl().RunInUnitTestMode(configuration);
 }
 
-int FuzzTestExternalEngineAdaptor::RunInFuzzingMode(
+bool FuzzTestExternalEngineAdaptor::RunInFuzzingMode(
     int* argc, char*** argv, const Configuration& configuration) {
   FUZZTEST_INTERNAL_CHECK(&LLVMFuzzerRunDriver,
                           "LibFuzzer Driver API not defined.");
@@ -101,7 +101,7 @@ int FuzzTestExternalEngineAdaptor::RunInFuzzingMode(
                           "Invalid fixture driver!");
   impl.fixture_driver_->TearDownFuzzTest();
 
-  return 0;
+  return true;
 }
 
 // External engine callbacks.

diff --git a/fuzztest/internal/compatibility_mode.h b/fuzztest/internal/compatibility_mode.h
@@ -75,9 +75,9 @@ class FuzzTestExternalEngineAdaptor : public FuzzTestFuzzer,
 
   FuzzTestExternalEngineAdaptor(const FuzzTest& test,
                                 std::unique_ptr<Driver> fixture_driver);
-  void RunInUnitTestMode(const Configuration& configuration) override;
-  int RunInFuzzingMode(int* argc, char*** argv,
-                       const Configuration& configuration) override;
+  bool RunInUnitTestMode(const Configuration& configuration) override;
+  bool RunInFuzzingMode(int* argc, char*** argv,
+                        const Configuration& configuration) override;
 
   // External engine callbacks.