Hi thestig@,

This PDF opens OK in Preview on my Mac. Can you move it along to the right PDF plug-in person to diagnose further?

Marking assigned to thestig@ and CCing tsepez@ for routing or assignment.

The issue seems to have to do with the second trailer. If you remove the second trailer in the document it opens correctly.

Looking at this a bit more, the original trailer was a V5 trailer. When we parse it we end up getting object 2 pointing to position 12. The new trailer isn't V4 or V5 so we go into ParseCrossRef. When that finishes we don't find an object 2 because one doesn't exist in the document.

The Pages entry says the page is at object 2.

So, the question becomes, when parsing the second trailer, why do we parse incorrectly.

Wei, can you take a look?

FWIW, Ghostscript's pdf2ps successfully converted the file to PS. Libpoppler-based PDF viewers cannot open the PDF either.

I will take a look. This pdf is weirdly formed though.

The following revision refers to this bug:
  https://pdfium.googlesource.com/pdfium.git/+/cd1e9ff4f432cbc29ed279e6891fb7ddc2ea3734

commit cd1e9ff4f432cbc29ed279e6891fb7ddc2ea3734
Author: weili <weili@chromium.org>
Date: Fri Apr 29 17:24:02 2016

Relax a couple checks to allow certain non-standard PDF files.

Some non-standard PDF files misuse the size of cross reference table,
and reuse some object number which the old one is still in use. PDFium
can relax the reusing of xref objects only since it is not referred in
the pdf document. When the size of cross reference table is larger
than defined, PDFium will try to continue other than abort.

BUG= chromium:596947 

Review-Url: https://codereview.chromium.org/1926823002

[modify] https://crrev.com/cd1e9ff4f432cbc29ed279e6891fb7ddc2ea3734/core/fpdfapi/fpdf_parser/cpdf_indirect_object_holder.cpp
[modify] https://crrev.com/cd1e9ff4f432cbc29ed279e6891fb7ddc2ea3734/core/fpdfapi/fpdf_parser/cpdf_parser.cpp
[modify] https://crrev.com/cd1e9ff4f432cbc29ed279e6891fb7ddc2ea3734/core/fpdfapi/fpdf_parser/cpdf_parser_embeddertest.cpp
[add] https://crrev.com/cd1e9ff4f432cbc29ed279e6891fb7ddc2ea3734/testing/resources/bug_596947.pdf

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a7b49810b22ebaed37a9290809dd6f21ae6bf453

commit a7b49810b22ebaed37a9290809dd6f21ae6bf453
Author: ochang <ochang@chromium.org>
Date: Mon May 02 20:06:38 2016

Roll PDFium 76c5379..a86d113

https://pdfium.googlesource.com/pdfium.git/+log/76c5379..a86d113

BUG=62400,589955, 607625 , 596947 , 602505 , 607739 

TBR=dsinclair@chromium.org

Review-Url: https://codereview.chromium.org/1938013002
Cr-Commit-Position: refs/heads/master@{#391037}

[modify] https://crrev.com/a7b49810b22ebaed37a9290809dd6f21ae6bf453/DEPS

The following revision refers to this bug:
  https://pdfium.googlesource.com/pdfium.git/+/5fc4f31285c3a88fc157fd2d9b9cf2eb5c7cabed

commit 5fc4f31285c3a88fc157fd2d9b9cf2eb5c7cabed
Author: weili <weili@chromium.org>
Date: Tue May 03 18:46:17 2016

Revert of Relax a couple checks to allow certain non-standard PDF files. (patchset #1 id:1 of https://codereview.chromium.org/1926823002/ )

Reason for revert:
Speculatively revert due to high volume of crashes on Chromium.

Original issue's description:
> Relax a couple checks to allow certain non-standard PDF files.
>
> Some non-standard PDF files misuse the size of cross reference table,
> and reuse some object number which the old one is still in use. PDFium
> can relax the reusing of xref objects only since it is not referred in
> the pdf document. When the size of cross reference table is larger
> than defined, PDFium will try to continue other than abort.
>
> BUG= chromium:596947 
>
> Committed: https://pdfium.googlesource.com/pdfium/+/cd1e9ff4f432cbc29ed279e6891fb7ddc2ea3734

TBR=thestig@chromium.org,dsinclair@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG= chromium:596947 

Review-Url: https://codereview.chromium.org/1946693002

[modify] https://crrev.com/5fc4f31285c3a88fc157fd2d9b9cf2eb5c7cabed/core/fpdfapi/fpdf_parser/cpdf_indirect_object_holder.cpp
[modify] https://crrev.com/5fc4f31285c3a88fc157fd2d9b9cf2eb5c7cabed/core/fpdfapi/fpdf_parser/cpdf_parser.cpp
[modify] https://crrev.com/5fc4f31285c3a88fc157fd2d9b9cf2eb5c7cabed/core/fpdfapi/fpdf_parser/cpdf_parser_embeddertest.cpp
[delete] https://crrev.com/fd670fdef8984fd7f8824b17d8fcf52fdec9aacc/testing/resources/bug_596947.pdf

The following revision refers to this bug:
  https://pdfium.googlesource.com/pdfium.git/+/a031357eaab7c934ac03717968cf78ff556c819b

commit a031357eaab7c934ac03717968cf78ff556c819b
Author: weili <weili@chromium.org>
Date: Wed May 04 16:36:11 2016

Reland of lax a couple checks to allow certain non-standard PDF files. (patchset #1 id:1 of https://codereview.chromium.org/1946693002/ )

Reason for revert:
The culprit was found and confirmed, not this one.

Original issue's description:
> Revert of Relax a couple checks to allow certain non-standard PDF files. (patchset #1 id:1 of https://codereview.chromium.org/1926823002/ )
>
> Reason for revert:
> Speculatively revert due to high volume of crashes on Chromium.
>
> Original issue's description:
> > Relax a couple checks to allow certain non-standard PDF files.
> >
> > Some non-standard PDF files misuse the size of cross reference table,
> > and reuse some object number which the old one is still in use. PDFium
> > can relax the reusing of xref objects only since it is not referred in
> > the pdf document. When the size of cross reference table is larger
> > than defined, PDFium will try to continue other than abort.
> >
> > BUG= chromium:596947 
> >
> > Committed: https://pdfium.googlesource.com/pdfium/+/cd1e9ff4f432cbc29ed279e6891fb7ddc2ea3734
>
> TBR=thestig@chromium.org,dsinclair@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG= chromium:596947 
>
> Committed: https://pdfium.googlesource.com/pdfium/+/5fc4f31285c3a88fc157fd2d9b9cf2eb5c7cabed

TBR=thestig@chromium.org,dsinclair@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG= chromium:596947 

Review-Url: https://codereview.chromium.org/1947983002

[modify] https://crrev.com/a031357eaab7c934ac03717968cf78ff556c819b/core/fpdfapi/fpdf_parser/cpdf_indirect_object_holder.cpp
[modify] https://crrev.com/a031357eaab7c934ac03717968cf78ff556c819b/core/fpdfapi/fpdf_parser/cpdf_parser.cpp
[modify] https://crrev.com/a031357eaab7c934ac03717968cf78ff556c819b/core/fpdfapi/fpdf_parser/cpdf_parser_embeddertest.cpp
[add] https://crrev.com/a031357eaab7c934ac03717968cf78ff556c819b/testing/resources/bug_596947.pdf

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/0b9ff5b70be2313da55f5d5cc450732b74d68dba

commit 0b9ff5b70be2313da55f5d5cc450732b74d68dba
Author: ochang <ochang@chromium.org>
Date: Mon May 09 17:23:13 2016

Roll PDFium 696fc8b..e00f75c

https://pdfium.googlesource.com/pdfium.git/+log/696fc8b..e00f75c

BUG= 596947 , 610236 

TBR=dsinclair@chromium.org
TEST=bots

Review-Url: https://codereview.chromium.org/1962743002
Cr-Commit-Position: refs/heads/master@{#392349}

[modify] https://crrev.com/0b9ff5b70be2313da55f5d5cc450732b74d68dba/DEPS

Reopening because the fix got reverted.

Hello!
This bug is receiving this notice because there has been no acknowledgment of its existence in quite a bit of time
- If you are currently working on this bug, please provide an update.
- If you are currently affected by this bug, please update with your current symptoms and relevant logs.

If there has been no updates provided by EOD Wednesday, 12/12/18 (5pm EST), this bug will be archived and can be re-opened at any time deemed necessary.

Thank you!

Due to lack of action this bug has been Archived. If work is still being done on this issue or you are still experiencing this issue please feel free to re-open with the appropriate information.