Don't pass null to memcmp.

src/core/SkPathRef.cpp

 /*
  * Copyright 2013 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkBuffer.h"
 #include "SkLazyPtr.h"
 #include "SkPath.h"
@@ skipping 175 matching lines @@
 #ifdef SK_RELEASE
     if (genIDMatch) {
         return true;
     }
 #endif
     if (fPointCnt != ref.fPointCnt ||
         fVerbCnt != ref.fVerbCnt) {
         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))) {
         SkASSERT(!genIDMatch);
         return false;
     }
     if (fConicWeights != ref.fConicWeights) {
         SkASSERT(!genIDMatch);
         return false;
     }
@@ skipping 271 matching lines @@
                 break;
             default:
                 SkDEBUGFAIL("Unknown Verb");
                 break;
         }
     }
     SkASSERT(mask == fSegmentMask);
 #endif // SK_DEBUG_PATH
 }
 #endif