commit 39cbe46df601fea51fc302b28b0423dc529fa027
author Kevin Lubick <kjlubick@google.com> Mon Mar 11 15:38:00 2019 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Mon Mar 11 20:08:07 2019 +0000
tree 907e8c2faf6dbfb9868b33e6085220d2dcdb8d93
parent e6266a2d16d31094044516b7d9f8dbdd8b5e5f4f

Add input length check when fuzzing

Otherwise, the string constructor can walk off the end
looking for a null terminator that never arrives.

Fix some logging copypasta

Bug: skia:
Change-Id: I0cb1b0b75673f64a5ac647307dbc04253f707686
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/199937
Reviewed-by: Kevin Lubick <kjlubick@google.com>
Reviewed-by: Ethan Nicholas <ethannicholas@google.com>
Commit-Queue: Kevin Lubick <kjlubick@google.com>

fuzz/oss_fuzz/FuzzSKSL2GLSL.cpp

diff --git a/fuzz/oss_fuzz/FuzzSKSL2GLSL.cpp b/fuzz/oss_fuzz/FuzzSKSL2GLSL.cpp
index ba4a0a0..9beca53 100644
--- a/fuzz/oss_fuzz/FuzzSKSL2GLSL.cpp
+++ b/fuzz/oss_fuzz/FuzzSKSL2GLSL.cpp
@@ -18,7 +18,8 @@
     settings.fCaps = caps.get();
     std::unique_ptr<SkSL::Program> program = compiler.convertProgram(
                                                     SkSL::Program::kFragment_Kind,
-                                                    SkSL::String((const char*) bytes->data()),
+                                                    SkSL::String((const char*) bytes->data(),
+                                                                 bytes->size()),
                                                     settings);
     if (!program || !compiler.toGLSL(*program, &output)) {
         return false;