diff options
| author |  Ivan Krasin <krasin@google.com> | 2016-11-18 18:28:30 -0800 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2016-11-19 06:47:34 +0000 |
| commit | 866f286c627fc56e6fea1dff82b645b238b75d20 (patch) | |
| tree | 3089c3654b24ae7c81540c071dea31d829cac1c5 /src | |
| parent | a675910aea8168c7face9ce28cac838b20b604bb (diff) | |
Avoid unnecessary cast on a garbage data.
It's dangerous and rightfully makes Control Flow Integrity check unhappy.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=5074
Change-Id: Ie59490a5c671240128a9fcb353108f67f32d2f9e
Reviewed-on: https://skia-review.googlesource.com/5074
Reviewed-by: Mike Klein <mtklein@chromium.org>
Commit-Queue: Mike Klein <mtklein@chromium.org>
Diffstat (limited to 'src')
| -rw-r--r-- | src/core/SkFixedAlloc.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/core/SkFixedAlloc.h b/src/core/SkFixedAlloc.h index 8d55cc7222..cbf5ac25d4 100644 --- a/src/core/SkFixedAlloc.h +++ b/src/core/SkFixedAlloc.h @@ -36,7 +36,7 @@ public: fUsed += skip; // Make space for T. - auto ptr = (T*)(fBuffer+fUsed); + void* ptr = fBuffer+fUsed; fUsed += sizeof(T); // Stamp a footer after the T that we can use to clean it up. |