diff options
| author |  Mike Klein <mtklein@chromium.org> | 2017-01-19 17:35:32 -0500 |
|---|---|---|
| committer | Mike Klein <mtklein@chromium.org> | 2017-01-19 22:44:13 +0000 |
| commit | 0d7d8de00a3ce2e70cf1221392db3d960610ad62 (patch) | |
| tree | c86b3fa7aaca63c14b53b03712d46a6a5777f6bf | |
| parent | de4d301881e7fd084f1f0b359ec6f9b2bf8bd4c5 (diff) | |
SkSplicer: always initialize fLowp
UBSAN sees copying non-bool (uninitialized) values into this bool. Not
really a big deal, as we'll never actually use it in that case, but it's
best to just initialize it to true or false.
Change-Id: I38137d46a8aa6e6651325002b71ae5e33ff24c4a
Reviewed-on: https://skia-review.googlesource.com/7321
Reviewed-by: Mike Klein <mtklein@chromium.org>
Commit-Queue: Mike Klein <mtklein@chromium.org>
| -rw-r--r-- | src/splicer/SkSplicer.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/splicer/SkSplicer.cpp b/src/splicer/SkSplicer.cpp index 57ce670a23..b289871576 100644 --- a/src/splicer/SkSplicer.cpp +++ b/src/splicer/SkSplicer.cpp @@ -302,6 +302,7 @@ namespace { fBackup = SkOpts::compile_pipeline(stages, nstages); fSplicedLen = 0; fSpliced = nullptr; + fLowp = false; // If we return early anywhere in here, !fSpliced means we'll use fBackup instead. #if defined(__aarch64__) |