diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2018-03-07 18:48:37 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-03-07 18:48:37 +0000 |
| commit | 749b609d11e855edf0aefdacbe4f81bb73d8d0d0 (patch) | |
| tree | c269fd9a4a1e374f52d7da611bfa39a6440cb2b4 /xfa/fxfa/cxfa_ffwidgethandler.cpp | |
| parent | 3f4befb2622487f3fe915ead4f78c7e3b940dec3 (diff) | |
| download | pdfium-749b609d11e855edf0aefdacbe4f81bb73d8d0d0.tar.xz | |
[formcalc] Handle bad elseif conditionalschromium/3365
This Cl adds checking for the conditionals of if and elseif expressions.
If the conditional fails to parse we should return nullptr. This already
happens by accident in the if() case, but with elseif() conditions we'll
fail the ASSERT in the CXFA_FMIfExpression constructor and crash.
This CL explicitly checks for the expressions and early exists if they
failed to parse.
Bug: chromium:819509
Change-Id: I9a90182c7709c8c4c0d3ae17d6be67cb668c0c6a
Reviewed-on: https://pdfium-review.googlesource.com/28131
Commit-Queue: dsinclair <dsinclair@chromium.org>
Commit-Queue: Ryan Harrison <rharrison@chromium.org>
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Reviewed-by: Ryan Harrison <rharrison@chromium.org>
Diffstat (limited to 'xfa/fxfa/cxfa_ffwidgethandler.cpp')
0 files changed, 0 insertions, 0 deletions