Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Value for /Type in Document Catalog not validated #210

Closed
asciim0 opened this issue Apr 1, 2017 · 0 comments · Fixed by #322
Closed

Value for /Type in Document Catalog not validated #210

asciim0 opened this issue Apr 1, 2017 · 0 comments · Fixed by #322
Assignees
@carlwilson
Labels
bug A product defect that needs fixing
Milestone
Release v1.20

Comments

@asciim0
Copy link
Contributor

asciim0 commented Apr 1, 2017

The document catalog can be found via an indirect reference in the trailer dictionary in key /Root. Per standard, the document catalog must have a /Type key of value /Catalog. T02-01_006 has a trailer pointing to obj 5 as the document catalog. While all other required entries for the Catalog dictionary are in place, the /Type value for the Catalog has been changed from the expected /Catalog to /Font. As a result the file cannot be rendered by Adobe Acrobat but JHOVE considers it well-formed and valid. There should be a consistency check for the indirect reference to the document catalog.
@ghost ghost added the bug A product defect that needs fixing label Mar 9, 2018
@ghost ghost added this to the Release v1.20 milestone Mar 9, 2018
@carlwilson carlwilson self-assigned this Mar 14, 2018
carlwilson added a commit that referenced this issue Mar 16, 2018
@carlwilson
FIX - Document catalog type checking
fe61f5d 
- added test for document catalog `Type` key to `PdfModule`;
- added `MessageConstants` entries for catalog type checking errors;
- unit tests and test resources for functionality.
Closes #210
@carlwilson carlwilson mentioned this issue Mar 16, 2018
Merged
@carlwilson carlwilson mentioned this issue Mar 19, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@carlwilson carlwilson

Labels
bug A product defect that needs fixing
Projects
None yet
Milestone
Release v1.20
Development

Successfully merging a pull request may close this issue.

FIX - PDF module, various processing errors openpreserve/jhove
2 participants
@carlwilson @asciim0