fuzz: fixing utility_fuzzer initialize_and_validate error handling (#…

…37316) Signed-off-by: Adi Suissa-Peleg <[email protected]>
envoyproxy · Nov 26, 2024 · 868d371 · 868d371
1 parent 4aa1d1b
commit 868d371
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/test/common/http/utility_corpus/empty_initialize_and_validate b/test/common/http/utility_corpus/empty_initialize_and_validate
diff --git a/test/common/http/utility_fuzz_test.cc b/test/common/http/utility_fuzz_test.cc
@@ -87,7 +87,7 @@ DEFINE_PROTO_FUZZER(const test::common::http::UtilityTestCase& input) {
   case test::common::http::UtilityTestCase::kInitializeAndValidate: {
     const auto& options = input.initialize_and_validate();
     auto options_or_error = Http2::Utility::initializeAndValidateOptions(options);
-    if (options_or_error.status().ok()) {
+    if (!options_or_error.status().ok()) {
       absl::string_view msg = options_or_error.status().message();
       // initializeAndValidateOptions throws exceptions for 4 different reasons due to malformed
       // settings, so check for them and allow any other exceptions through