diff options
author | Cary Clark <caryclark@skia.org> | 2017-07-05 12:09:16 -0400 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2017-07-05 19:11:39 +0000 |
commit | 4bba683632b1db429343ca6cc54f803ac857ef35 (patch) | |
tree | 795d80ac551eda18e14abda5830f683126d72f4e /core/fxge/skia | |
parent | 1b4670cab5be6a5938621a25f050da7351c507b2 (diff) | |
download | pdfium-4bba683632b1db429343ca6cc54f803ac857ef35.tar.xz |
clip save off by one
Code around this bug was altered in April
to fix the off by one bug, but somehow the
loop counter fix was overlooked.
R=dsinclair@chromium.org,npm@chromium.org
Bug: 736195
Change-Id: I583a9f2389e6111ae1b847b961afaafd5e854810
Reviewed-on: https://pdfium-review.googlesource.com/7276
Reviewed-by: dsinclair <dsinclair@chromium.org>
Commit-Queue: Cary Clark <caryclark@google.com>
Diffstat (limited to 'core/fxge/skia')
-rw-r--r-- | core/fxge/skia/fx_skia_device.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/core/fxge/skia/fx_skia_device.cpp b/core/fxge/skia/fx_skia_device.cpp index 7e0d55688c..8e9dfa6126 100644 --- a/core/fxge/skia/fx_skia_device.cpp +++ b/core/fxge/skia/fx_skia_device.cpp @@ -933,7 +933,7 @@ class SkiaState { return false; Dump(__func__); int count = m_commands.count(); - if (m_commandIndex < count - 1) { + if (m_commandIndex < count) { if (Clip::kSave == m_commands[m_commandIndex]) { ++m_commandIndex; return true; |