diff options
| author |  caryclark <caryclark@google.com> | 2016-10-20 08:32:18 -0700 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2016-10-20 08:32:18 -0700 |
| commit | a35ab3e6e024d0b548ded26a2e3b8ecd838ead93 (patch) | |
| tree | c009069f86b1129fcf5037ffc8e8fbf1b9ba010f /src/pathops/SkIntersections.h | |
| parent | 65820db5e15201a3f30968420232d36c0ca89cd8 (diff) | |
fix fuzzers
Many old pathops-related fuzz failures have built up while
the codebase was under a state a flux. Now that the code
is stable, address these failures.
Most of the CL plumbs the debug global state to downstream
routines so that, if the data is not trusted (ala fuzzed)
the function can safely exit without asserting.
TBR=reed@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2426173002
Review-Url: https://chromiumcodereview.appspot.com/2426173002
Diffstat (limited to 'src/pathops/SkIntersections.h')
| -rw-r--r-- | src/pathops/SkIntersections.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/pathops/SkIntersections.h b/src/pathops/SkIntersections.h index abc10e19dd..d5d217cc80 100644 --- a/src/pathops/SkIntersections.h +++ b/src/pathops/SkIntersections.h @@ -104,7 +104,7 @@ public: } #ifdef SK_DEBUG - SkOpGlobalState* debugGlobalState() { return fDebugGlobalState; } + SkOpGlobalState* globalState() const { return fDebugGlobalState; } #endif bool hasT(double t) const { @@ -308,9 +308,9 @@ private: void cleanUpParallelLines(bool parallel); void computePoints(const SkDLine& line, int used); - SkDPoint fPt[12]; // FIXME: since scans store points as SkPoint, this should also + SkDPoint fPt[13]; // FIXME: since scans store points as SkPoint, this should also SkDPoint fPt2[2]; // used by nearly same to store alternate intersection point - double fT[2][12]; + double fT[2][13]; uint16_t fIsCoincident[2]; // bit set for each curve's coincident T bool fNearlySame[2]; // true if end points nearly match unsigned char fUsed; |