[Zucchini] Remove imposed_ensemble_matcher_fuzzer size check

This check isn't very helpful. It only finds pathological but valid
situations. Typically, the resulting patch is very compressible and
this only checks the uncompressed size, which is inflated due to having
many headers for ZTF and Raw regions.

Bug: 860845
Change-Id: I5747e787a1c9888c4ef70d8449572517669f39cd
Reviewed-on: https://chromium-review.googlesource.com/1128810
Commit-Queue: Calder Kitagawa <ckitagawa@chromium.org>
Reviewed-by: Samuel Huang <huangs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#573372}
NOKEYCHECK=True
GitOrigin-RevId: 71ac85257f083f1d3ceded06f1d66992e033d389

1 files changed, 1 insertions, 6 deletions

diff --git a/fuzzers/imposed_ensemble_matcher_fuzzer.cc b/fuzzers/imposed_ensemble_matcher_fuzzer.cc
index bbb06ca..0dbcf86 100644
--- a/fuzzers/imposed_ensemble_matcher_fuzzer.cc
+++ b/fuzzers/imposed_ensemble_matcher_fuzzer.cc
@@ -59,13 +59,8 @@ DEFINE_BINARY_PROTO_FUZZER(const zucchini::fuzzers::FilePair& file_pair) {
   zucchini::GenerateBufferImposed(old_image, new_image,
                                   file_pair.imposed_matches(), &patch_writer);
 
-  // Check that the patch size is sane. Crash the fuzzer if this isn't the case,
-  // as it is a failure in Zucchini's patch performance that is worth
-  // investigating.
-  size_t patch_size = patch_writer.SerializedSize();
-  CHECK_LE(patch_size, kMaxImageSize * 3);
-
   // Write to buffer to avoid IO.
+  size_t patch_size = patch_writer.SerializedSize();
   std::unique_ptr<uint8_t[]> patch_data(new uint8_t[patch_size]);
   zucchini::BufferSink patch(patch_data.get(), patch_size);
   patch_writer.SerializeInto(patch);