diff options
author | Mike Klein <mtklein@chromium.org> | 2018-05-23 12:09:41 -0400 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-05-23 16:31:11 +0000 |
commit | 8ed1a8942693f0c516006eaae21a3c00def6c9a5 (patch) | |
tree | 81373cd2224a713ee7a22570fa65660a3df291c5 /fuzz | |
parent | c494a708be977a458a105318b2e2f439ab985ed8 (diff) |
remove SkColorSpace::MakeICC() fuzzer
We're removing SkColorSpace::MakeICC() and replacing it with skcms.
The equivalent skcms_Parse() is already fuzzed.
Change-Id: Ic5c8f8c71976859b9892fdc6991aea90dac04eff
Reviewed-on: https://skia-review.googlesource.com/129652
Auto-Submit: Mike Klein <mtklein@chromium.org>
Commit-Queue: Kevin Lubick <kjlubick@google.com>
Reviewed-by: Kevin Lubick <kjlubick@google.com>
Diffstat (limited to 'fuzz')
-rwxr-xr-x | fuzz/coverage | 4 | ||||
-rw-r--r-- | fuzz/fuzz.cpp | 15 |
2 files changed, 2 insertions, 17 deletions
diff --git a/fuzz/coverage b/fuzz/coverage index 87ba5bea37..d681d9369a 100755 --- a/fuzz/coverage +++ b/fuzz/coverage @@ -50,9 +50,9 @@ GCOV="$(realpath tools/gcov_shim)" lcov -q --gcov-tool="$GCOV" -c -b "$BUILD" -d "$BUILD" -o "$DIR"/baseline -i # Running the binary generates the real coverage information, the .gcda files. -QUEUES=("$1/api_parse_path/fuzzer0/queue/*" "$1/color_deserialize/fuzzer0/queue/*" "$1/color_icc/fuzzer0/queue/*" "$1/skcodec_scale/fuzzer0/queue/*" "$1/skcodec_mode/fuzzer0/queue/*" "$1/api_draw_functions/fuzzer0/queue/*" "$1/api_gradient/fuzzer0/queue/*" "$1/api_image_filter/fuzzer0/queue/*" "$1/api_pathop/fuzzer0/queue/*" "$1/sksl2glsl/fuzzer0/queue/*" "$1/null_canvas/fuzzer0/queue/*" "$1/pdf_canvas/fuzzer0/queue/*" "$1/n32_canvas/fuzzer0/queue/*") +QUEUES=("$1/api_parse_path/fuzzer0/queue/*" "$1/color_deserialize/fuzzer0/queue/*" "$1/skcodec_scale/fuzzer0/queue/*" "$1/skcodec_mode/fuzzer0/queue/*" "$1/api_draw_functions/fuzzer0/queue/*" "$1/api_gradient/fuzzer0/queue/*" "$1/api_image_filter/fuzzer0/queue/*" "$1/api_pathop/fuzzer0/queue/*" "$1/sksl2glsl/fuzzer0/queue/*" "$1/null_canvas/fuzzer0/queue/*" "$1/pdf_canvas/fuzzer0/queue/*" "$1/n32_canvas/fuzzer0/queue/*") -ARGS=("-n ParsePath" "-t color_deserialize" "-t icc" "-t image_scale" "-t image_mode" "-n DrawFunctions" "-n Gradients" "-n SerializedImageFilter" "-n Pathop" "-t sksl2glsl" "-n NullCanvas" "-n PDFCanvas" "-n RasterN32Canvas") +ARGS=("-n ParsePath" "-t color_deserialize" "-t image_scale" "-t image_mode" "-n DrawFunctions" "-n Gradients" "-n SerializedImageFilter" "-n Pathop" "-t sksl2glsl" "-n NullCanvas" "-n PDFCanvas" "-n RasterN32Canvas") # We can't simply pass the directories to the fuzzers because some of the fuzzes will # crash or assert, which would kill the call to fuzz prematurely. Instead we run them diff --git a/fuzz/fuzz.cpp b/fuzz/fuzz.cpp index 9857e80178..161969fb43 100644 --- a/fuzz/fuzz.cpp +++ b/fuzz/fuzz.cpp @@ -52,7 +52,6 @@ DEFINE_string2(type, t, "", "How to interpret --bytes, one of:\n" "api\n" "color_deserialize\n" "filter_fuzz (equivalent to Chrome's filter_fuzz_stub)\n" - "icc\n" "image_decode\n" "image_mode\n" "image_scale\n" @@ -72,7 +71,6 @@ static SkString try_auto_detect(SkString path, SkString* name); static void fuzz_api(sk_sp<SkData> bytes, SkString name); static void fuzz_color_deserialize(sk_sp<SkData>); static void fuzz_filter_fuzz(sk_sp<SkData>); -static void fuzz_icc(sk_sp<SkData>); static void fuzz_img2(sk_sp<SkData>); static void fuzz_animated_img(sk_sp<SkData>); static void fuzz_img(sk_sp<SkData>, uint8_t, uint8_t); @@ -149,10 +147,6 @@ static int fuzz_file(SkString path, SkString type) { fuzz_color_deserialize(bytes); return 0; } - if (type.equals("icc")) { - fuzz_icc(bytes); - return 0; - } if (type.equals("image_decode")) { fuzz_img2(bytes); return 0; @@ -618,15 +612,6 @@ static void fuzz_skpipe(sk_sp<SkData> bytes) { SkDebugf("[terminated] Success! Decoded and rendered an SkPicture from SkPipe!\n"); } -static void fuzz_icc(sk_sp<SkData> bytes) { - sk_sp<SkColorSpace> space(SkColorSpace::MakeICC(bytes->data(), bytes->size())); - if (!space) { - SkDebugf("[terminated] Couldn't decode ICC.\n"); - return; - } - SkDebugf("[terminated] Success! Decoded ICC.\n"); -} - static void fuzz_color_deserialize(sk_sp<SkData> bytes) { sk_sp<SkColorSpace> space(SkColorSpace::Deserialize(bytes->data(), bytes->size())); if (!space) { |