commit bd94ccb9bdbc1530db3b61ebbd5deb4d356bd1bc
author bnc <bnc@google.com> Wed Oct 20 09:18:49 2021 -0700
committer Copybara-Service <copybara-worker@google.com> Wed Oct 20 09:20:07 2021 -0700
tree 9d64b94ae878c752a5f1711e59082d634b4f72d0
parent d05e8834b09739323c36ecd78642082204d2d07e

Exercise Flush() in QpackEncoderStreamSender fuzzer.

Also, remove TODO to feed data to a QpackEncoderStreamReceiver with a mock
Delegate and add EXPECT_CALLs for each instruction.  Mock objects are expensive
to instantiate, and fuzzer targets need to be very fast in order to be
effective.  QpackEncoderStreamReceiver is fuzzed by its own fuzzer.

PiperOrigin-RevId: 404563168

quic/core/qpack/fuzzer/qpack_encoder_stream_sender_fuzzer.cc

diff --git a/quic/core/qpack/fuzzer/qpack_encoder_stream_sender_fuzzer.cc b/quic/core/qpack/fuzzer/qpack_encoder_stream_sender_fuzzer.cc
index 1281ebf..ab0e083 100644
--- a/quic/core/qpack/fuzzer/qpack_encoder_stream_sender_fuzzer.cc
+++ b/quic/core/qpack/fuzzer/qpack_encoder_stream_sender_fuzzer.cc
@@ -17,9 +17,6 @@
 namespace test {
 
 // This fuzzer exercises QpackEncoderStreamSender.
-// TODO(bnc): Encoded data could be fed into QpackEncoderStreamReceiver and
-// decoded instructions directly compared to input.  Figure out how to get gMock
-// enabled for cc_fuzz_target target types.
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t* data, size_t size) {
   NoopQpackStreamSenderDelegate delegate;
   QpackEncoderStreamSender sender;
@@ -30,7 +27,7 @@
   const uint16_t kMaxStringLength = 2048;
 
   while (provider.remaining_bytes() != 0) {
-    switch (provider.ConsumeIntegral<uint8_t>() % 4) {
+    switch (provider.ConsumeIntegral<uint8_t>() % 5) {
       case 0: {
         bool is_static = provider.ConsumeBool();
         uint64_t name_index = provider.ConsumeIntegral<uint64_t>();
@@ -61,9 +58,14 @@
         sender.SendSetDynamicTableCapacity(capacity);
         break;
       }
+      case 4: {
+        sender.Flush();
+        break;
+      }
     }
   }
 
+  sender.Flush();
   return 0;
 }