Fix exit code when an input other than the primary input generates warnings

m-holger
1 parent 0046de08
Showing 3 changed files with 8 additions and 1 deletions
libqpdf/QPDFJob.cc
qpdf/qtest/page-errors.test
qpdf/qtest/qpdf/page-missing-mediabox.out
@@ -477,6 +477,12 @@ QPDFJob::createQPDF()
     }
     handleUnderOverlay(pdf);
     handleTransformations(pdf);
+
+    for (auto& foreign: page_heap) {
+        if (foreign->anyWarnings()) {
+            m->warnings = true;
+        }
+    }
     return pdf_sp;
 }
  
@@ -22,7 +22,7 @@ $td-&gt;runtest(&quot;handle page no with contents&quot;,
              $td->NORMALIZE_NEWLINES);
 $td->runtest("handle page with missing MediaBox",
              {$td->COMMAND => "qpdf --static-id --empty --pages page-no-content.pdf -- out.pdf"},
-             {$td->FILE => "page-missing-mediabox.out", $td->EXIT_STATUS => 0},
+             {$td->FILE => "page-missing-mediabox.out", $td->EXIT_STATUS => 3},
              $td->NORMALIZE_NEWLINES);
 $td->runtest("check output",
              {$td->FILE => "out.pdf"},
 WARNING: page-no-content.pdf, object 4 0 at offset 288: kid 1 (from 0) MediaBox is undefined; setting to letter / ANSI A
+qpdf: operation succeeded with warnings; resulting file may have some problems