Don't pass null to memcmp.

src/core/SkPathRef.cpp

Index: src/core/SkPathRef.cpp
diff --git a/src/core/SkPathRef.cpp b/src/core/SkPathRef.cpp
index dffb6a3646aefd27727ff357d63f855e4e4247b0..61763c1b068c92577b4bac815d5112d66e5ea2e2 100644
--- a/src/core/SkPathRef.cpp
+++ b/src/core/SkPathRef.cpp
@@ -193,12 +193,17 @@ bool SkPathRef::operator== (const SkPathRef& ref) const {
         SkASSERT(!genIDMatch);
         return false;
     }
+    if (ref.fVerbCnt == 0 && ref.fPointCnt == 0) {

[reed1, 2014/11/14 17:09:41]

is it enough to just check fVerbCnt (and if it is zero, assert that fPointCnt is
also zero)?

[mtklein, 2014/11/14 17:13:20]

Yes (until we add pointless verbs :P).  Done.

+        return true;
+    }
+    SkASSERT(this->verbsMemBegin() && ref.verbsMemBegin());
     if (0 != memcmp(this->verbsMemBegin(),
                     ref.verbsMemBegin(),
                     ref.fVerbCnt * sizeof(uint8_t))) {
         SkASSERT(!genIDMatch);
         return false;
     }
+    SkASSERT(this->points() && ref.points());
     if (0 != memcmp(this->points(),
                     ref.points(),
                     ref.fPointCnt * sizeof(SkPoint))) {