diff options
| author | Wei Li <weili@chromium.org> | 2017-03-23 09:45:04 -0700 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-03-23 17:05:05 +0000 |
| commit | 6bdd824188bc9a2e6b24b5752a3170ce10185c1d (patch) | |
| tree | 272c426e10e66315ef2993d2d8712029aa6a90b5 /xfa/fxfa/app/xfa_ffdocview.cpp | |
| parent | 409b663d532d4d6f09a1188fa3b9ac4044708bc4 (diff) | |
| download | pdfium-6bdd824188bc9a2e6b24b5752a3170ce10185c1d.tar.xz | |
Fix two CloneNonCycle issues
CloneNonCycle() tries to detect cyclic object references without copying
them. There are two issues:
-- for elements in an array or a dictionary, they should be able to
refer to the same object, which are not cyclic;
-- for cyclic referenced elements in an array or a dictionary, do not
clone the element at all. Having nullptr or <key, nullptr> as an element,
like we did before, might cause crash when the element being accessed.
BUG=chromium:701860
Change-Id: Id0304accde76ed06fa5ce640994c7628359600fb
Reviewed-on: https://pdfium-review.googlesource.com/3156
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fxfa/app/xfa_ffdocview.cpp')
0 files changed, 0 insertions, 0 deletions