diff options
| author |  Hal Canary <halcanary@google.com> | 2018-04-06 10:25:12 -0400 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-04-06 16:37:23 +0000 |
| commit | 13872ddc56c4ab128569428b3ea8c722e7494df5 (patch) | |
| tree | 9cc10636ced3893c720ebc08ef8b600a06bd34f2 /fuzz/FuzzCommon.h | |
| parent | 4c2393493ad6d5f72eecd1190fee20cce732b959 (diff) | |
Region Op Fuzzer
Also: assert Region IRects don't contain Sentinel value.
Also: Region fuzzer can't produce Sentinel value.
Change-Id: Ia33c7eeb9ef2c46b3da4a025d85de8a0406c3c0c
Reviewed-on: https://skia-review.googlesource.com/119011
Reviewed-by: Kevin Lubick <kjlubick@google.com>
Reviewed-by: Cary Clark <caryclark@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
Diffstat (limited to 'fuzz/FuzzCommon.h')
| -rw-r--r-- | fuzz/FuzzCommon.h | 26 |
1 files changed, 19 insertions, 7 deletions
diff --git a/fuzz/FuzzCommon.h b/fuzz/FuzzCommon.h index bea5a30f86..7615fe723e 100644 --- a/fuzz/FuzzCommon.h +++ b/fuzz/FuzzCommon.h @@ -26,22 +26,34 @@ inline void fuzz_nice_float(Fuzz* fuzz, float* f, Args... rest) { fuzz_nice_float(fuzz, rest...); } -template <> -inline void Fuzz::next(SkRegion* region) { +template <typename T, typename Min, typename Max> +inline void fuzz_enum_range(Fuzz* fuzz, T* value, Min rmin, Max rmax) { + using U = skstd::underlying_type_t<T>; + fuzz->nextRange((U*)value, (U)rmin, (U)rmax); +} + +inline void fuzz_region(Fuzz* fuzz, SkRegion* region, int maxN) { uint8_t N; - this->nextRange(&N, 0, 10); + fuzz->nextRange(&N, 0, maxN); for (uint8_t i = 0; i < N; ++i) { SkIRect r; - uint8_t op; - this->next(&r); + SkRegion::Op op; + // Avoid the sentinal value used by Region. + fuzz->nextRange(&r.fLeft, -2147483646, 2147483646); + fuzz->nextRange(&r.fTop, -2147483646, 2147483646); + fuzz->nextRange(&r.fRight, -2147483646, 2147483646); + fuzz->nextRange(&r.fBottom, -2147483646, 2147483646); r.sort(); - this->nextRange(&op, 0, (uint8_t)SkRegion::kLastOp); - if (!region->op(r, (SkRegion::Op)op)) { + fuzz_enum_range(fuzz, &op, (SkRegion::Op)0, SkRegion::kLastOp); + if (!region->op(r, op)) { return; } } } +template <> +inline void Fuzz::next(SkRegion* region) { fuzz_region(this, region, 10); } + // allows some float values for path points void FuzzPath(Fuzz* fuzz, SkPath* path, int maxOps); // allows all float values for path points |