Add client HTTP/2 fuzzer

Signed-off-by: Yan Avlasov <yavlasov@google.com>
yanavlasov · Jun 30, 2023 · 1ba6c48 · 1ba6c48
1 parent 766b290
commit 1ba6c48
Show file tree

Hide file tree

Showing 7 changed files with 397 additions and 106 deletions.
diff --git a/test/common/http/BUILD b/test/common/http/BUILD
@@ -644,6 +644,7 @@ envoy_cc_fuzz_test(
         "//source/common/stream_info:stream_info_lib",
         "//source/common/upstream:upstream_includes",
         "//source/common/upstream:upstream_lib",
+        "//test/common/http/http2:http2_frame",
         "//test/common/upstream:utility_lib",
         "//test/config:utility_lib",
         "//test/fuzz:utility_lib",

diff --git a/test/common/http/client_codec_diff_fuzz_test.cc b/test/common/http/client_codec_diff_fuzz_test.cc
diff --git a/test/common/http/http2/http2_frame.cc b/test/common/http/http2/http2_frame.cc
@@ -19,7 +19,7 @@ namespace Envoy {
 namespace Http {
 namespace Http2 {
 
-const char Http2Frame::Preamble[25] = "PRI * HTTP/2.0\r\n\r\nSM\r\n\r\n";
+const absl::string_view Http2Frame::Preamble("PRI * HTTP/2.0\r\n\r\nSM\r\n\r\n");
 
 void Http2Frame::setHeader(absl::string_view header) {
   ASSERT(header.size() >= HeaderSize);

diff --git a/test/common/http/http2/http2_frame.h b/test/common/http/http2/http2_frame.h
@@ -50,7 +50,7 @@ class Http2Frame {
   using ConstIterator = DataContainer::const_iterator;
 
   static constexpr size_t HeaderSize = 9;
-  static const char Preamble[25];
+  static const absl::string_view Preamble;
 
   enum class Type : uint8_t {
     Data = 0,

diff --git a/test/common/http/server_codec_diff_fuzz_corpus/kitchen_sink_content_length_only b/test/common/http/server_codec_diff_fuzz_corpus/kitchen_sink_content_length_only
diff --git a/test/common/http/server_codec_diff_fuzz_corpus/redirect_with_content_length b/test/common/http/server_codec_diff_fuzz_corpus/redirect_with_content_length
diff --git a/test/common/http/server_codec_diff_fuzz_test.cc b/test/common/http/server_codec_diff_fuzz_test.cc
@@ -735,7 +735,7 @@ class Http2HcmTest : public HcmTest {
     }
 
     for (const auto& header : input_.request().headers()) {
-      absl::string_view value = header.value();
+      std::string value = header.value();
       if (input_.send_request_body() && header.key() == "content-length") {
         value = std::to_string(request_body.size());
       }