diff options
author | Kevin Lubick <kjlubick@google.com> | 2018-02-01 09:45:11 -0500 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-02-01 15:10:43 +0000 |
commit | a71b8d17e8b0cc1ee19a75d75c1f52dc8030f68d (patch) | |
tree | 443551da1485f1169c8c48585e84823c6030fce4 /fuzz/oss_fuzz | |
parent | 08895c48144cedaf81006803afe4a5a2becfdb92 (diff) |
Check for nullptrs when fuzzing region_deserialize
Bug: oss-fuzz:5629
Change-Id: I1129a6a9a68c69e07ab63e2e2be1c00cf0581962
Reviewed-on: https://skia-review.googlesource.com/102482
Reviewed-by: Mike Klein <mtklein@chromium.org>
Commit-Queue: Kevin Lubick <kjlubick@google.com>
Diffstat (limited to 'fuzz/oss_fuzz')
-rw-r--r-- | fuzz/oss_fuzz/FuzzRegionDeserialize.cpp | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/fuzz/oss_fuzz/FuzzRegionDeserialize.cpp b/fuzz/oss_fuzz/FuzzRegionDeserialize.cpp index c5b37cb68e..80ee4f95ae 100644 --- a/fuzz/oss_fuzz/FuzzRegionDeserialize.cpp +++ b/fuzz/oss_fuzz/FuzzRegionDeserialize.cpp @@ -24,7 +24,11 @@ bool FuzzRegionDeserialize(sk_sp<SkData> bytes) { } else { region.contains(1,1); } - auto s = SkSurface::MakeRasterN32Premul(1024, 1024); + auto s = SkSurface::MakeRasterN32Premul(128, 128); + if (!s) { + // May return nullptr in memory-constrained fuzzing environments + return false; + } s->getCanvas()->drawRegion(region, SkPaint()); SkDEBUGCODE(region.validate()); return true; |