summaryrefslogtreecommitdiff
path: root/fpdfsdk/formfiller/cffl_checkbox.cpp
diff options
context:
space:
mode:
authorLuật Nguyễn <manhluat93.php@gmail.com>2017-10-10 12:39:22 +0800
committerChromium commit bot <commit-bot@chromium.org>2017-10-10 20:12:46 +0000
commit1886471c3432dee4d9a9be5678a757dde8717652 (patch)
tree437dd29735be2351e475819798e40edddf41b50b /fpdfsdk/formfiller/cffl_checkbox.cpp
parentdd002931a16a99b0c6e6ae7b6cba9d4dafb27e18 (diff)
downloadpdfium-1886471c3432dee4d9a9be5678a757dde8717652.tar.xz
Fix UAF in SaveData on all of CFFL_* types.
Bug: 756427 Change-Id: I8e31d96c6f3b83a6464ed69c95225362c50386d1 Reviewed-on: https://pdfium-review.googlesource.com/15870 Commit-Queue: Tom Sepez <tsepez@chromium.org> Reviewed-by: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'fpdfsdk/formfiller/cffl_checkbox.cpp')
-rw-r--r--fpdfsdk/formfiller/cffl_checkbox.cpp6
1 files changed, 6 insertions, 0 deletions
diff --git a/fpdfsdk/formfiller/cffl_checkbox.cpp b/fpdfsdk/formfiller/cffl_checkbox.cpp
index 2863a56001..e9c72efd39 100644
--- a/fpdfsdk/formfiller/cffl_checkbox.cpp
+++ b/fpdfsdk/formfiller/cffl_checkbox.cpp
@@ -111,9 +111,15 @@ void CFFL_CheckBox::SaveData(CPDFSDK_PageView* pPageView) {
}
}
}
+ CPDFSDK_Widget::ObservedPtr observed_widget(m_pWidget.Get());
+ CFFL_CheckBox::ObservedPtr observed_this(this);
m_pWidget->SetCheck(bNewChecked, false);
+ if (!observed_widget)
+ return;
m_pWidget->UpdateField();
+ if (!observed_widget || !observed_this)
+ return;
SetChangeMark();
}