Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(224)

Issue 1504723002: Merge to XFA: fix for stream object reading (Closed)

Created:
5 years ago by Wei Li
Modified:
5 years ago
Reviewers:
jun_fang
CC:
pdfium-reviews_googlegroups.com
Base URL:
https://pdfium.googlesource.com/pdfium.git@xfa
Target Ref:
refs/heads/xfa
Visibility:
Public.

Description

Merge to XFA: fix for stream object reading Loosen a check for earlier version of PDF files. When the bytes with specified length are followed by 'endstream' keyword, even if there is no EOL marker before the keyword, it signals the end of stream. BUG=551258 TBR=jun_fang@foxitsoftware.com Review URL: https://codereview.chromium.org/1499433002 . (cherry picked from commit 0ff66089c87ab6e3adaaff0ec69728ce7a8d8299) Committed: https://pdfium.googlesource.com/pdfium/+/63da7474adf5b585d5fd10729a99a2c50c20a626

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+64 lines, -4 lines) Patch
M core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp View 1 chunk +4 lines, -4 lines 0 comments Download
A testing/resources/pixel/bug_551258_1.in View 1 chunk +60 lines, -0 lines 0 comments Download
A + testing/resources/pixel/bug_551258_1_expected.pdf.0.png View Binary file 0 comments Download
A + testing/resources/pixel/bug_551258_1_expected_mac.pdf.0.png View Binary file 0 comments Download

Messages

Total messages: 3 (1 generated)
Wei Li
TBR, clean merge
5 years ago (2015-12-07 04:23:52 UTC) #1
Wei Li
5 years ago (2015-12-07 04:24:29 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
63da7474adf5b585d5fd10729a99a2c50c20a626 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698