aboutsummaryrefslogtreecommitdiffstats
path: root/include
diff options
context:
space:
mode:
authorJay Berkenbilt <ejb@ql.org>2018-03-06 17:25:09 +0100
committerJay Berkenbilt <ejb@ql.org>2018-03-06 17:34:07 +0100
commite4e2e26d990d038b0d35e7466c8a24dbfafab7d2 (patch)
tree3f9341102bdadb710912b700e26b8be6e2288123 /include
parent1a4dcb4aaf987631caa93a9d3a5b42ca439bb6ae (diff)
downloadqpdf-e4e2e26d990d038b0d35e7466c8a24dbfafab7d2.tar.zst
Properly handle pages with no contents (fixes #194)
Remove calls to assertPageObject(). All cases in the library that called assertPageObject() work fine if you don't call assertPageObject() because nothing assumes anything that was being checked by that call. Removing the calls enables more files to be successfully processed.
Diffstat (limited to 'include')
-rw-r--r--include/qpdf/QPDFObjectHandle.hh5
1 files changed, 5 insertions, 0 deletions
diff --git a/include/qpdf/QPDFObjectHandle.hh b/include/qpdf/QPDFObjectHandle.hh
index a2f54a73..6f78e9a9 100644
--- a/include/qpdf/QPDFObjectHandle.hh
+++ b/include/qpdf/QPDFObjectHandle.hh
@@ -830,6 +830,11 @@ class QPDFObjectHandle
QPDF_DLL
void assertNumber();
+ // The isPageObject method checks the /Type key of the object.
+ // This is not completely reliable as there are some otherwise
+ // valid files whose /Type is wrong for page objects. qpdf is
+ // slightly more accepting but may still return false here when
+ // treating the object as a page would work. Use this sparingly.
QPDF_DLL
bool isPageObject();
QPDF_DLL