summaryrefslogtreecommitdiff
path: root/core/fxge
diff options
context:
space:
mode:
authorCary Clark <caryclark@skia.org>2017-07-05 12:09:16 -0400
committerChromium commit bot <commit-bot@chromium.org>2017-07-05 19:11:39 +0000
commit4bba683632b1db429343ca6cc54f803ac857ef35 (patch)
tree795d80ac551eda18e14abda5830f683126d72f4e /core/fxge
parent1b4670cab5be6a5938621a25f050da7351c507b2 (diff)
downloadpdfium-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')
-rw-r--r--core/fxge/skia/fx_skia_device.cpp2
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;