Handle pages with no /Contents from getPageContents()

The spec allows /Contents to be omitted for pages that are blank, but QPDFObjectHandle::getPageContents() was throwing an exception in this case.

Handle pages with no /Contents from getPageContents()
The spec allows /Contents to be omitted for pages that are blank, but QPDFObjectHandle::getPageContents() was throwing an exception in this case.
Jay Berkenbilt
1 parent 4071db59
Showing 5 changed files with 23 additions and 2 deletions
ChangeLog
libqpdf/QPDFObjectHandle.cc
qpdf/qtest/qpdf.test
qpdf/qtest/qpdf/page-no-content.out
qpdf/qtest/qpdf/page-no-content.pdf
+2014-11-14  Jay Berkenbilt  <ejb@ql.org>
+
+	* Bug fix: QPDFObjectHandle::getPageContents() no longer throws an
+	exception when called on a page that has no /Contents key in its
+	dictionary. This is allowed by the spec, and some software
+	packages generate files like this for pages that are blank in the
+	original.
+
 2014-06-07  Jay Berkenbilt  <ejb@ql.org>
 	* 5.1.2: release
@@ -633,7 +633,7 @@ QPDFObjectHandle::getPageContents()
     {
 	result.push_back(contents);
     }
-    else
+    else if (! contents.isNull())
     {
 	throw std::runtime_error("unknown object type inspecting /Contents "
 				 "key in page dictionary");
@@ -199,7 +199,7 @@ $td-&gt;runtest(&quot;remove page we don&#39;t have&quot;,
 show_ntests();
 # ----------
 $td->notify("--- Miscellaneous Tests ---");
-$n_tests += 72;
+$n_tests += 73;
 $td->runtest("qpdf version",
 	     {$td->COMMAND => "qpdf --version"},
@@ -554,6 +554,11 @@ $td-&gt;runtest(&quot;check file&quot;,
              {$td->FILE => "a.pdf"},
              {$td->FILE => "indirect-decode-parms-out.pdf"});
+$td->runtest("handle page no with contents",
+             {$td->COMMAND => "qpdf --show-pages page-no-content.pdf"},
+             {$td->FILE => "page-no-content.out", $td->EXIT_STATUS => 0},
+             $td->NORMALIZE_NEWLINES);
+
 show_ntests();
 # ----------
 $td->notify("--- Numeric range parsing tests ---");
+page 1: 3 0 R
+  content:
+    6 0 R
+page 2: 4 0 R
+  content:
+page 3: 5 0 R
+  content:
+    9 0 R