replace max_len with if statements

Zepeng Hu · Skia Commit-Bot · commit f1eb43e8800b · 2020-06-16T17:26:30.000Z
Change-Id: I60d60e9b9ea0b7b6544a36bef7f4e263bb9de532 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/296416 Reviewed-by: Kevin Lubick <kjlubick@google.com> Commit-Queue: Zepeng Hu <zepenghu@google.com>
diff --git a/fuzz/oss_fuzz/FuzzAPIImageFilter.cpp b/fuzz/oss_fuzz/FuzzAPIImageFilter.cpp
@@ -10,6 +10,9 @@
 void fuzz_ImageFilter(Fuzz* f);
 
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 4000) {
+        return 0;
+    }
     auto fuzz = Fuzz(SkData::MakeWithoutCopy(data, size));
     fuzz_ImageFilter(&fuzz);
     return 0;
diff --git a/fuzz/oss_fuzz/FuzzAPISkDescriptor.cpp b/fuzz/oss_fuzz/FuzzAPISkDescriptor.cpp
@@ -11,6 +11,9 @@ void fuzz_SkDescriptor(Fuzz* f);
 
 #if defined(IS_FUZZING_WITH_LIBFUZZER)
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 4000) {
+        return 0;
+    }
     auto fuzz = Fuzz(SkData::MakeWithoutCopy(data, size));
     fuzz_SkDescriptor(&fuzz);
     return 0;
diff --git a/fuzz/oss_fuzz/FuzzAndroidCodec.cpp b/fuzz/oss_fuzz/FuzzAndroidCodec.cpp
@@ -52,6 +52,9 @@ bool FuzzAndroidCodec(sk_sp<SkData> bytes, uint8_t sampleSize) {
 
 #if defined(IS_FUZZING_WITH_LIBFUZZER)
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 10240) {
+        return 0;
+    }
     auto bytes = SkData::MakeWithoutCopy(data, size);
     Fuzz fuzz(bytes);
     uint8_t sampleSize;
diff --git a/fuzz/oss_fuzz/FuzzAnimatedImage.cpp b/fuzz/oss_fuzz/FuzzAnimatedImage.cpp
@@ -38,6 +38,9 @@ bool FuzzAnimatedImage(sk_sp<SkData> bytes) {
 
 #if defined(IS_FUZZING_WITH_LIBFUZZER)
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 10240) {
+        return 0;
+    }
     auto bytes = SkData::MakeWithoutCopy(data, size);
     FuzzAnimatedImage(bytes);
     return 0;
diff --git a/fuzz/oss_fuzz/FuzzDrawFunctions.cpp b/fuzz/oss_fuzz/FuzzDrawFunctions.cpp
@@ -10,6 +10,9 @@
 void fuzz_DrawFunctions(Fuzz* f);
 
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 4000) {
+        return 0;
+    }
     auto fuzz = Fuzz(SkData::MakeWithoutCopy(data, size));
     fuzz_DrawFunctions(&fuzz);
     return 0;
diff --git a/fuzz/oss_fuzz/FuzzGradients.cpp b/fuzz/oss_fuzz/FuzzGradients.cpp
@@ -10,6 +10,9 @@
 void fuzz_Gradients(Fuzz* f);
 
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 4000) {
+        return 0;
+    }
     auto fuzz = Fuzz(SkData::MakeWithoutCopy(data, size));
     fuzz_Gradients(&fuzz);
     return 0;
diff --git a/fuzz/oss_fuzz/FuzzImage.cpp b/fuzz/oss_fuzz/FuzzImage.cpp
@@ -30,6 +30,9 @@ bool FuzzImageDecode(sk_sp<SkData> bytes) {
 
 #if defined(IS_FUZZING_WITH_LIBFUZZER)
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 10240) {
+        return 0;
+    }
     auto bytes = SkData::MakeWithoutCopy(data, size);
     FuzzImageDecode(bytes);
     return 0;
diff --git a/fuzz/oss_fuzz/FuzzImageFilterDeserialize.cpp b/fuzz/oss_fuzz/FuzzImageFilterDeserialize.cpp
@@ -41,6 +41,9 @@ void FuzzImageFilterDeserialize(sk_sp<SkData> bytes) {
 
 #if defined(IS_FUZZING_WITH_LIBFUZZER)
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 10024) {
+        return 0;
+    }
     gSkFontMgr_DefaultFactory = &ToolUtils::MakePortableFontMgr;
     auto bytes = SkData::MakeWithoutCopy(data, size);
     FuzzImageFilterDeserialize(bytes);
diff --git a/fuzz/oss_fuzz/FuzzIncrementalImage.cpp b/fuzz/oss_fuzz/FuzzIncrementalImage.cpp
@@ -47,6 +47,9 @@ bool FuzzIncrementalImageDecode(sk_sp<SkData> bytes) {
 
 #if defined(IS_FUZZING_WITH_LIBFUZZER)
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 10240) {
+        return 0;
+    }
     auto bytes = SkData::MakeWithoutCopy(data, size);
     FuzzIncrementalImageDecode(bytes);
     return 0;
diff --git a/fuzz/oss_fuzz/FuzzJPEGEncoder.cpp b/fuzz/oss_fuzz/FuzzJPEGEncoder.cpp
@@ -10,6 +10,9 @@
 void fuzz_JPEGEncoder(Fuzz* f);
 
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 262150) {
+        return 0;
+    }
     auto fuzz = Fuzz(SkData::MakeWithoutCopy(data, size));
     fuzz_JPEGEncoder(&fuzz);
     return 0;
diff --git a/fuzz/oss_fuzz/FuzzMockGPUCanvas.cpp b/fuzz/oss_fuzz/FuzzMockGPUCanvas.cpp
@@ -20,6 +20,9 @@ extern "C" {
     }
 
     int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+        if (size > 4000) {
+            return 0;
+        }
         gSkFontMgr_DefaultFactory = &ToolUtils::MakePortableFontMgr;
         auto fuzz = Fuzz(SkData::MakeWithoutCopy(data, size));
         fuzz_MockGPUCanvas(&fuzz);
diff --git a/fuzz/oss_fuzz/FuzzNullCanvas.cpp b/fuzz/oss_fuzz/FuzzNullCanvas.cpp
@@ -12,6 +12,9 @@
 void fuzz_NullCanvas(Fuzz* f);
 
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 4000) {
+        return 0;
+    }
     gSkFontMgr_DefaultFactory = &ToolUtils::MakePortableFontMgr;
     auto fuzz = Fuzz(SkData::MakeWithoutCopy(data, size));
     fuzz_NullCanvas(&fuzz);
diff --git a/fuzz/oss_fuzz/FuzzPNGEncoder.cpp b/fuzz/oss_fuzz/FuzzPNGEncoder.cpp
@@ -10,6 +10,9 @@
 void fuzz_PNGEncoder(Fuzz* f);
 
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 262150) {
+        return 0;
+    }
     auto fuzz = Fuzz(SkData::MakeWithoutCopy(data, size));
     fuzz_PNGEncoder(&fuzz);
     return 0;
diff --git a/fuzz/oss_fuzz/FuzzPathDeserialize.cpp b/fuzz/oss_fuzz/FuzzPathDeserialize.cpp
@@ -28,7 +28,7 @@ void FuzzPathDeserialize(SkReadBuffer& buf) {
 
 #if defined(IS_FUZZING_WITH_LIBFUZZER)
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
-    if (size < 4) {
+    if (size < 4 || size > 2000) {
         return 0;
     }
     uint32_t packed;
diff --git a/fuzz/oss_fuzz/FuzzPathMeasure.cpp b/fuzz/oss_fuzz/FuzzPathMeasure.cpp
@@ -10,6 +10,9 @@
 void fuzz_PathMeasure(Fuzz* f);
 
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 4000) {
+        return 0;
+    }
     auto fuzz = Fuzz(SkData::MakeWithoutCopy(data, size));
     fuzz_PathMeasure(&fuzz);
     return 0;
diff --git a/fuzz/oss_fuzz/FuzzPathop.cpp b/fuzz/oss_fuzz/FuzzPathop.cpp
@@ -10,6 +10,9 @@
 void fuzz_Pathop(Fuzz* f);
 
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 4000) {
+        return 0;
+    }
     auto fuzz = Fuzz(SkData::MakeWithoutCopy(data, size));
     fuzz_Pathop(&fuzz);
     return 0;
diff --git a/fuzz/oss_fuzz/FuzzPolyUtils.cpp b/fuzz/oss_fuzz/FuzzPolyUtils.cpp
@@ -10,6 +10,9 @@
 void fuzz_PolyUtils(Fuzz* f);
 
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 4000) {
+        return 0;
+    }
     auto fuzz = Fuzz(SkData::MakeWithoutCopy(data, size));
     fuzz_PolyUtils(&fuzz);
     return 0;
diff --git a/fuzz/oss_fuzz/FuzzRasterN32Canvas.cpp b/fuzz/oss_fuzz/FuzzRasterN32Canvas.cpp
@@ -12,6 +12,9 @@
 void fuzz_RasterN32Canvas(Fuzz* f);
 
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 4000) {
+        return 0;
+    }
     gSkFontMgr_DefaultFactory = &ToolUtils::MakePortableFontMgr;
     auto fuzz = Fuzz(SkData::MakeWithoutCopy(data, size));
     fuzz_RasterN32Canvas(&fuzz);
diff --git a/fuzz/oss_fuzz/FuzzRegionDeserialize.cpp b/fuzz/oss_fuzz/FuzzRegionDeserialize.cpp
@@ -36,6 +36,9 @@ bool FuzzRegionDeserialize(sk_sp<SkData> bytes) {
 
 #if defined(IS_FUZZING_WITH_LIBFUZZER)
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 512) {
+        return 0;
+    }
     auto bytes = SkData::MakeWithoutCopy(data, size);
     FuzzRegionDeserialize(bytes);
     return 0;
diff --git a/fuzz/oss_fuzz/FuzzRegionSetPath.cpp b/fuzz/oss_fuzz/FuzzRegionSetPath.cpp
@@ -38,6 +38,9 @@ void FuzzRegionSetPath(Fuzz* fuzz) {
 
 #if defined(IS_FUZZING_WITH_LIBFUZZER)
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 512) {
+        return 0;
+    }
     sk_sp<SkData> bytes(SkData::MakeWithoutCopy(data, size));
     Fuzz fuzz(bytes);
     FuzzRegionSetPath(&fuzz);
diff --git a/fuzz/oss_fuzz/FuzzSKSL2GLSL.cpp b/fuzz/oss_fuzz/FuzzSKSL2GLSL.cpp
@@ -29,6 +29,9 @@ bool FuzzSKSL2GLSL(sk_sp<SkData> bytes) {
 
 #if defined(IS_FUZZING_WITH_LIBFUZZER)
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 3000) {
+        return 0;
+    }
     auto bytes = SkData::MakeWithoutCopy(data, size);
     FuzzSKSL2GLSL(bytes);
     return 0;
diff --git a/fuzz/oss_fuzz/FuzzSKSL2Metal.cpp b/fuzz/oss_fuzz/FuzzSKSL2Metal.cpp
@@ -29,6 +29,9 @@ bool FuzzSKSL2Metal(sk_sp<SkData> bytes) {
 
 #if defined(IS_FUZZING_WITH_LIBFUZZER)
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 3000) {
+        return 0;
+    }
     auto bytes = SkData::MakeWithoutCopy(data, size);
     FuzzSKSL2Metal(bytes);
     return 0;
diff --git a/fuzz/oss_fuzz/FuzzSKSL2Pipeline.cpp b/fuzz/oss_fuzz/FuzzSKSL2Pipeline.cpp
@@ -29,6 +29,9 @@ bool FuzzSKSL2Pipeline(sk_sp<SkData> bytes) {
 
 #if defined(IS_FUZZING_WITH_LIBFUZZER)
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 3000) {
+        return 0;
+    }
     auto bytes = SkData::MakeWithoutCopy(data, size);
     FuzzSKSL2Pipeline(bytes);
     return 0;
diff --git a/fuzz/oss_fuzz/FuzzSKSL2SPIRV.cpp b/fuzz/oss_fuzz/FuzzSKSL2SPIRV.cpp
@@ -29,6 +29,9 @@ bool FuzzSKSL2SPIRV(sk_sp<SkData> bytes) {
 
 #if defined(IS_FUZZING_WITH_LIBFUZZER)
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 3000) {
+        return 0;
+    }
     auto bytes = SkData::MakeWithoutCopy(data, size);
     FuzzSKSL2SPIRV(bytes);
     return 0;
diff --git a/fuzz/oss_fuzz/FuzzSVG.cpp b/fuzz/oss_fuzz/FuzzSVG.cpp
@@ -33,6 +33,9 @@ void FuzzSVG(sk_sp<SkData> bytes) {
 
 #if defined(IS_FUZZING_WITH_LIBFUZZER)
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 30000) {
+        return 0;
+    }
     auto bytes = SkData::MakeWithoutCopy(data, size);
     FuzzSVG(bytes);
     return 0;
diff --git a/fuzz/oss_fuzz/FuzzSkDescriptorDeserialize.cpp b/fuzz/oss_fuzz/FuzzSkDescriptorDeserialize.cpp
@@ -29,6 +29,9 @@ void FuzzSkDescriptorDeserialize(sk_sp<SkData> bytes) {
 
 #if defined(IS_FUZZING_WITH_LIBFUZZER)
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 1024) {
+        return 0;
+    }
     auto bytes = SkData::MakeWithoutCopy(data, size);
     FuzzSkDescriptorDeserialize(bytes);
     return 0;
diff --git a/fuzz/oss_fuzz/FuzzTextBlobDeserialize.cpp b/fuzz/oss_fuzz/FuzzTextBlobDeserialize.cpp
@@ -29,6 +29,9 @@ void FuzzTextBlobDeserialize(SkReadBuffer& buf) {
 
 #if defined(IS_FUZZING_WITH_LIBFUZZER)
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 1024) {
+        return 0;
+    }
     gSkFontMgr_DefaultFactory = &ToolUtils::MakePortableFontMgr;
     SkReadBuffer buf(data, size);
     FuzzTextBlobDeserialize(buf);
diff --git a/fuzz/oss_fuzz/FuzzWEBPEncoder.cpp b/fuzz/oss_fuzz/FuzzWEBPEncoder.cpp
@@ -10,6 +10,9 @@
 void fuzz_WEBPEncoder(Fuzz* f);
 
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
+    if (size > 262150) {
+        return 0;
+    }
     auto fuzz = Fuzz(SkData::MakeWithoutCopy(data, size));
     fuzz_WEBPEncoder(&fuzz);
     return 0;
