diff options
| author | Lei Zhang <thestig@chromium.org> | 2017-06-13 14:04:02 -0700 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-06-15 18:28:02 +0000 |
| commit | 6a3fc45b9e238d9b7b601cb13be664391d393b42 (patch) | |
| tree | 69182b8517f5ecea3ebe24f46fa72b43d70a7f26 /fpdfsdk/formfiller/cffl_radiobutton.cpp | |
| parent | 65a55343e623924c9c3bbbd953097cf7fd0f5fc6 (diff) | |
| download | pdfium-6a3fc45b9e238d9b7b601cb13be664391d393b42.tar.xz | |
Add more checks for destroyed annotations in CFFL_FormFiller.
CFFL_FormFiller::CommitData() should check more rigorously and so should
its callers.
BUG=chromium:732051
Change-Id: If0cee8fb61de10dc7678dad89c330d75bee55aa4
Reviewed-on: https://pdfium-review.googlesource.com/6530
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'fpdfsdk/formfiller/cffl_radiobutton.cpp')
| -rw-r--r-- | fpdfsdk/formfiller/cffl_radiobutton.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/fpdfsdk/formfiller/cffl_radiobutton.cpp b/fpdfsdk/formfiller/cffl_radiobutton.cpp index e5a5a98b9a..c6ce432d0c 100644 --- a/fpdfsdk/formfiller/cffl_radiobutton.cpp +++ b/fpdfsdk/formfiller/cffl_radiobutton.cpp @@ -60,8 +60,7 @@ bool CFFL_RadioButton::OnChar(CPDFSDK_Annot* pAnnot, if (CPWL_RadioButton* pWnd = (CPWL_RadioButton*)GetPDFWindow(pPageView, true)) pWnd->SetCheck(true); - CommitData(pPageView, nFlags); - return true; + return CommitData(pPageView, nFlags); } default: return CFFL_FormFiller::OnChar(pAnnot, nChar, nFlags); @@ -79,8 +78,7 @@ bool CFFL_RadioButton::OnLButtonUp(CPDFSDK_PageView* pPageView, (CPWL_RadioButton*)GetPDFWindow(pPageView, true)) pWnd->SetCheck(true); - if (!CommitData(pPageView, nFlags)) - return false; + return CommitData(pPageView, nFlags); } return true; |