Fix ImageFilter fuzzer issue

src/core/SkReadBuffer.h

Index: src/core/SkReadBuffer.h
diff --git a/src/core/SkReadBuffer.h b/src/core/SkReadBuffer.h
index faf853aef5430124c7f916636804c88fa6eb4df1..c5c4708feb50f1b48252086fb9f3493de17c86fc 100644
--- a/src/core/SkReadBuffer.h
+++ b/src/core/SkReadBuffer.h
@@ -101,6 +101,9 @@ public:
     size_t offset() { return fReader.offset(); }
     bool eof() { return fReader.eof(); }
     virtual const void* skip(size_t size) { return fReader.skip(size); }
+
+    void setOffset(size_t offset) { fReader.setOffset(offset); }

[reed1, 2016/04/20 13:26:37]

who needs this, why?

[robertphillips, 2016/04/20 17:09:40]

SkPicturePlayback::handleOp uses it as an optimization to skip to the restore
operation if everything is clipped out (for clipRect, clipPath, clipRRect and
clipRegion).

I have added a validating override.

+
     void* readFunctionPtr() { return fReader.readPtr(); }
 
     // primitives
@@ -121,6 +124,7 @@ public:
     virtual void readMatrix(SkMatrix* matrix);
     virtual void readIRect(SkIRect* rect);
     virtual void readRect(SkRect* rect);
+    virtual void readRRect(SkRRect* rrect);
     virtual void readRegion(SkRegion* region);
 
     virtual void readPath(SkPath* path);
@@ -203,9 +207,12 @@ public:
     }
 
     // Default impelementations don't check anything.
-    virtual bool validate(bool isValid) { return true; }
+    virtual bool validate(bool isValid) { return isValid; }
     virtual bool isValid() const { return true; }
     virtual bool validateAvailable(size_t size) { return true; }
+    bool validateIndex(int index, int count) {
+        return this->validate(index >= 0 && index < count);
+    }
 
 protected:
     SkReader32 fReader;