fuzz: fix h10 codec validation (envoyproxy#29924)

Signed-off-by: Adi Suissa-Peleg <[email protected]>
amolbms · Oct 3, 2023 · 9f8b7ea · 9f8b7ea
1 parent 99f7178
commit 9f8b7ea
Show file tree

Hide file tree

Showing 2 changed files with 39 additions and 0 deletions.
diff --git a/test/common/http/codec_impl_corpus/h10_empty_hostname b/test/common/http/codec_impl_corpus/h10_empty_hostname
diff --git a/test/common/http/codec_impl_fuzz_test.cc b/test/common/http/codec_impl_fuzz_test.cc
@@ -66,6 +66,12 @@ Http1Settings fromHttp1Settings(const test::common::http::Http1ServerSettings& s
   h1_settings.accept_http_10_ = settings.accept_http_10();
   h1_settings.default_host_for_http_10_ = settings.default_host_for_http_10();
 
+  // If the server accepts a HTTP/1.0 then the default host must be valid.
+  if (h1_settings.accept_http_10_ &&
+      !HeaderUtility::authorityIsValid(h1_settings.default_host_for_http_10_)) {
+    throw EnvoyException("Invalid Http1ServerSettings, HTTP/1.0 is enabled and "
+                         "'default_host_for_http_10' has invalid hostname, skipping test.");
+  }
   return h1_settings;
 }