Initial error handling code

src/effects/SkBlurImageFilter.cpp

Index: src/effects/SkBlurImageFilter.cpp
diff --git a/src/effects/SkBlurImageFilter.cpp b/src/effects/SkBlurImageFilter.cpp
index 3f97ddd59a362896d18e3083210068b934c42f10..051421df9d027f92c6d006f85ea9b216ec37b711 100644
--- a/src/effects/SkBlurImageFilter.cpp
+++ b/src/effects/SkBlurImageFilter.cpp
@@ -19,6 +19,12 @@ SkBlurImageFilter::SkBlurImageFilter(SkFlattenableReadBuffer& buffer)
   : INHERITED(buffer) {
     fSigma.fWidth = buffer.readScalar();
     fSigma.fHeight = buffer.readScalar();
+      if (!SkScalarIsFinite(fSigma.fWidth) ||
+          !SkScalarIsFinite(fSigma.fHeight) ||
+          (fSigma.fWidth < 0) ||
+          (fSigma.fHeight < 0)) {
+        buffer.setError();

[reed1, 2013/08/29 19:08:22]

Might add a helper for this pattern (now or later)

e.g.

buffer.setErrorIfTrue(predicate);

(with a better name)

[sugoi1, 2013/08/29 19:39:40]

Done. Called it validateData() for now.

+    }
 }
 
 SkBlurImageFilter::SkBlurImageFilter(SkScalar sigmaX,