From 0d7d8de00a3ce2e70cf1221392db3d960610ad62 Mon Sep 17 00:00:00 2001 From: Mike Klein Date: Thu, 19 Jan 2017 17:35:32 -0500 Subject: 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 Commit-Queue: Mike Klein --- src/splicer/SkSplicer.cpp | 1 + 1 file changed, 1 insertion(+) 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__) -- cgit v1.2.3