Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(172)

Issue 2099113004: Fix mask applied to SkPath::fFillType to fix fuzzer bug (Closed)

Created:
4 years, 5 months ago by robertphillips
Modified:
4 years, 5 months ago
Reviewers:
bsalomon
CC:
reviews_skia.org
Base URL:
https://skia.googlesource.com/skia.git@master
Target Ref:
refs/heads/master
Project:
skia
Visibility:
Public.

Description

Fix mask applied to SkPath::fFillType in readFromMemory to fix fuzzer bug The fFillType field only needs/uses 2 bits - not all 8 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2099113004 Committed: https://skia.googlesource.com/skia/+/7070a3c44b24da5715dc4da618cb014f87e92451

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M src/core/SkPath.cpp View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 14 (8 generated)
robertphillips
4 years, 5 months ago (2016-06-27 20:02:19 UTC) #5
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2099113004/1
4 years, 5 months ago (2016-06-27 20:03:35 UTC) #7
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
4 years, 5 months ago (2016-06-27 20:28:02 UTC) #9
bsalomon
lgtm
4 years, 5 months ago (2016-06-28 01:21:09 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2099113004/1
4 years, 5 months ago (2016-06-28 11:54:01 UTC) #12
commit-bot: I haz the power
4 years, 5 months ago (2016-06-28 11:54:57 UTC) #14
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/7070a3c44b24da5715dc4da618cb014f87e92451

Powered by Google App Engine
This is Rietveld 408576698