aboutsummaryrefslogtreecommitdiffhomepage
path: root/fuzz/oss_fuzz
diff options
context:
space:
mode:
authorGravatar Kevin Lubick <kjlubick@google.com>2018-02-01 09:45:11 -0500
committerGravatar Skia Commit-Bot <skia-commit-bot@chromium.org>2018-02-01 15:10:43 +0000
commita71b8d17e8b0cc1ee19a75d75c1f52dc8030f68d (patch)
tree443551da1485f1169c8c48585e84823c6030fce4 /fuzz/oss_fuzz
parent08895c48144cedaf81006803afe4a5a2becfdb92 (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.cpp6
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;