diff options
| author | Jun Fang <jun_fang@foxitsoftware.com> | 2015-09-25 20:32:46 -0700 |
|---|---|---|
| committer | Jun Fang <jun_fang@foxitsoftware.com> | 2015-09-25 20:32:46 -0700 |
| commit | 9bd18183ba8210c91d71c3060146235750a4c71c (patch) | |
| tree | e1950c29e9e295dd247a0edc5ed8d490e18800d8 /core/src/fpdfapi/fpdf_page/fpdf_page_parser.cpp | |
| parent | 163f59b9a0b019539e9a463ec183c964e7317d5b (diff) | |
| download | pdfium-9bd18183ba8210c91d71c3060146235750a4c71c.tar.xz | |
Fix the issue that pdfium swallows 'fi' or 'ff' in some pdf files
Pdfium swallows 'fi' or 'ff' in some tested files because it doesn't load the embedded font file correctly. The root cause is that there is incorrect keyword like 'ngendstream' in the stream of the embedded font file. Pdfium tries to find another correct keyword but uses wrong offset rather than accumulated offset.
BUG=524043
R=thestig@chromium.org, tsepez@chromium.org
Review URL: https://codereview.chromium.org/1307353005 .
Diffstat (limited to 'core/src/fpdfapi/fpdf_page/fpdf_page_parser.cpp')
| -rw-r--r-- | core/src/fpdfapi/fpdf_page/fpdf_page_parser.cpp | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/core/src/fpdfapi/fpdf_page/fpdf_page_parser.cpp b/core/src/fpdfapi/fpdf_page/fpdf_page_parser.cpp index 059dd4c2a6..4e5ef1c898 100644 --- a/core/src/fpdfapi/fpdf_page/fpdf_page_parser.cpp +++ b/core/src/fpdfapi/fpdf_page/fpdf_page_parser.cpp @@ -11,7 +11,6 @@ #define REQUIRE_PARAMS(count) \ if (m_ParamCount != count) { \ - m_bAbort = TRUE; \ return; \ } @@ -34,7 +33,6 @@ CPDF_StreamContentParser::CPDF_StreamContentParser( m_Level(level), m_ParamStartPos(0), m_ParamCount(0), - m_bAbort(FALSE), m_pCurStates(new CPDF_AllStates), m_pLastTextObject(nullptr), m_DefFontSize(0), |