diff options
| author | Lei Zhang <thestig@chromium.org> | 2017-08-02 17:41:22 -0700 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-08-03 01:05:51 +0000 |
| commit | d1a8458e6390103e123e9d265040b3d02c16955b (patch) | |
| tree | 3b169237c62b6b6210722ed403314b52aea34d8f /core/fpdfapi/parser/cpdf_data_avail.cpp | |
| parent | f44812cf6d7559dd2a0bbc25a684e54675f4794d (diff) | |
| download | pdfium-d1a8458e6390103e123e9d265040b3d02c16955b.tar.xz | |
Avoid a redundant header offset check in CPDF_Parser.
CPDF_Parser::StartLinearizedParse() calls StartParse(), but already
knows the PDF header offset. Refactor StartParse() so it does not have
to look for the header again.
Change-Id: Id8cc39301ae72da868dafc53921622d5b28ce26e
Reviewed-on: https://pdfium-review.googlesource.com/9830
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Art Snake <art-snake@yandex-team.ru>
Diffstat (limited to 'core/fpdfapi/parser/cpdf_data_avail.cpp')
| -rw-r--r-- | core/fpdfapi/parser/cpdf_data_avail.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/core/fpdfapi/parser/cpdf_data_avail.cpp b/core/fpdfapi/parser/cpdf_data_avail.cpp index 88755a9eb7..46c71a5ab9 100644 --- a/core/fpdfapi/parser/cpdf_data_avail.cpp +++ b/core/fpdfapi/parser/cpdf_data_avail.cpp @@ -754,7 +754,7 @@ bool CPDF_DataAvail::IsLinearizedFile(uint8_t* pData, uint32_t dwLen) { auto file = pdfium::MakeRetain<CFX_MemoryStream>( pData, static_cast<size_t>(dwLen), false); int32_t offset = GetHeaderOffset(file); - if (offset == -1) { + if (offset == kInvalidHeaderOffset) { m_docStatus = PDF_DATAAVAIL_ERROR; return false; } |