summaryrefslogtreecommitdiff
path: root/core
diff options
context:
space:
mode:
authorLei Zhang <thestig@chromium.org>2017-02-24 14:07:17 -0800
committerChromium commit bot <commit-bot@chromium.org>2017-02-27 15:58:45 +0000
commit717a4fc857d66017cecc4c8f8285713135b9dc68 (patch)
tree55816996a4907d87cacc02b4966d977d7629139d /core
parent1d7018e92d315c883dca38dd083db44791761827 (diff)
downloadpdfium-717a4fc857d66017cecc4c8f8285713135b9dc68.tar.xz
Remove some pointless checks and assignments.
Change-Id: I6f9fd67cdac581e8f91f829a5934b763bb229d21 Reviewed-on: https://pdfium-review.googlesource.com/2838 Commit-Queue: dsinclair <dsinclair@chromium.org> Reviewed-by: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'core')
-rw-r--r--core/fpdfapi/parser/cpdf_data_avail.cpp12
1 files changed, 2 insertions, 10 deletions
diff --git a/core/fpdfapi/parser/cpdf_data_avail.cpp b/core/fpdfapi/parser/cpdf_data_avail.cpp
index c6ecae9b1d..64eaf2a43b 100644
--- a/core/fpdfapi/parser/cpdf_data_avail.cpp
+++ b/core/fpdfapi/parser/cpdf_data_avail.cpp
@@ -1124,13 +1124,8 @@ bool CPDF_DataAvail::CheckArrayPageNode(uint32_t dwPageNo,
return false;
}
- if (!pPages) {
- if (m_docStatus == PDF_DATAAVAIL_ERROR) {
- m_docStatus = PDF_DATAAVAIL_ERROR;
- return false;
- }
+ if (!pPages)
return false;
- }
CPDF_Array* pArray = pPages->AsArray();
if (!pArray) {
@@ -1161,11 +1156,8 @@ bool CPDF_DataAvail::CheckUnknownPageNode(uint32_t dwPageNo,
return false;
}
- if (!pPage) {
- if (m_docStatus == PDF_DATAAVAIL_ERROR)
- m_docStatus = PDF_DATAAVAIL_ERROR;
+ if (!pPage)
return false;
- }
if (pPage->IsArray()) {
pPageNode->m_dwPageNo = dwPageNo;