summaryrefslogtreecommitdiff
path: root/core/src/fpdfapi
diff options
context:
space:
mode:
authorBruce Dawson <brucedawson@google.com>2014-12-23 11:43:16 -0800
committerBruce Dawson <brucedawson@google.com>2014-12-23 11:43:16 -0800
commitcf8e1b281f0ece4c23568aa9a01f259aa84836f1 (patch)
treea0ee7a7950e066ca66f437dad00a76366cca095c /core/src/fpdfapi
parente1082c90fa84db89810fd596c73db414f5a865d8 (diff)
downloadpdfium-cf8e1b281f0ece4c23568aa9a01f259aa84836f1.tar.xz
XFA: merge patch from CL 727083002, zero local variables.
Zero initialize ch to avoid possible bug - conditions are very subtle. Whether ch and iRet are read without being initialized depends on complex preconditions and cannot be determined by looking at these function. Therefore it seems prudent to zero initialize them to avoid any risk. BUG=427616 TBR=bo_xu@foxitsoftware.com Review URL: https://codereview.chromium.org/727083002 Review URL: https://codereview.chromium.org/793763005
Diffstat (limited to 'core/src/fpdfapi')
-rw-r--r--core/src/fpdfapi/fpdf_edit/fpdf_edit_create.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/core/src/fpdfapi/fpdf_edit/fpdf_edit_create.cpp b/core/src/fpdfapi/fpdf_edit/fpdf_edit_create.cpp
index fd94b88983..4a34de1c16 100644
--- a/core/src/fpdfapi/fpdf_edit/fpdf_edit_create.cpp
+++ b/core/src/fpdfapi/fpdf_edit/fpdf_edit_create.cpp
@@ -2084,7 +2084,7 @@ FX_INT32 CPDF_Creator::Continue(IFX_Pause *pPause)
if (m_iStage < 0) {
return m_iStage;
}
- FX_INT32 iRet;
+ FX_INT32 iRet = 0;
while (m_iStage < 100) {
if (m_iStage < 20) {
iRet = WriteDoc_Stage1(pPause);