From ebe79ffd8627e3fea945a82895e64cede4ec21bf Mon Sep 17 00:00:00 2001 From: Mike Klein Date: Tue, 6 Dec 2016 10:36:52 -0500 Subject: fStages.data() should always be defined, but UBSAN says NO CQ_INCLUDE_TRYBOTS=skia.primary:Test-Ubuntu-Clang-GCE-CPU-AVX2-x86_64-Debug-ASAN,Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD Change-Id: I6aef3ec2bcb85d6ad4830c759a1234ef2e31f32d Reviewed-on: https://skia-review.googlesource.com/5552 Commit-Queue: Mike Klein Reviewed-by: Mike Klein --- src/core/SkRasterPipeline.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/core/SkRasterPipeline.cpp b/src/core/SkRasterPipeline.cpp index 6d31a55f28..4c1dbc762e 100644 --- a/src/core/SkRasterPipeline.cpp +++ b/src/core/SkRasterPipeline.cpp @@ -20,7 +20,9 @@ void SkRasterPipeline::extend(const SkRasterPipeline& src) { } void SkRasterPipeline::run(size_t x, size_t y, size_t n) const { - SkOpts::run_pipeline(x,y,n, fStages.data(), SkToInt(fStages.size())); + if (!fStages.empty()) { + SkOpts::run_pipeline(x,y,n, fStages.data(), SkToInt(fStages.size())); + } } std::function SkRasterPipeline::compile() const { -- cgit v1.2.3