diff options
| author | Ryan Harrison <rharrison@chromium.org> | 2018-02-06 21:45:44 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-02-06 21:45:44 +0000 |
| commit | 0cdb8434d6113eb84a68b6e3505eb73934ef17ce (patch) | |
| tree | fc3763a2a4eac2c05e27d0435e35c8380f80402b /testing/xfa_js_embedder_test.cpp | |
| parent | bfeab62b37a701dc82f180e49d26c602f96f96f9 (diff) | |
| download | pdfium-0cdb8434d6113eb84a68b6e3505eb73934ef17ce.tar.xz | |
Use temporary iterator to avoid potential OOB
In the existing code pCharPos is manipulated directly without being
reset. This means that for the second iteration it is at the end of
the range instead of the start. This CL introduces temporary iterators
that are intialized to the value of pCharPos and then manipulated to
avoid this issue and having to reset pCharPos.
BUG=chromium:648177
Change-Id: I5c9344c1b67a015b01470a0dc337361552ffd447
Reviewed-on: https://pdfium-review.googlesource.com/25750
Commit-Queue: Ryan Harrison <rharrison@chromium.org>
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Diffstat (limited to 'testing/xfa_js_embedder_test.cpp')
0 files changed, 0 insertions, 0 deletions