From 20a134826c6f33fab81b0cdb9ba2d75fb03d1b59 Mon Sep 17 00:00:00 2001 From: Jay Berkenbilt Date: Tue, 9 Jan 2024 08:35:10 -0500 Subject: Fix arg parsing check with --collate The check for the number of items was in the wrong place. --- libqpdf/QPDFJob.cc | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'libqpdf/QPDFJob.cc') diff --git a/libqpdf/QPDFJob.cc b/libqpdf/QPDFJob.cc index 2e0abb43..3bdb4f39 100644 --- a/libqpdf/QPDFJob.cc +++ b/libqpdf/QPDFJob.cc @@ -2446,6 +2446,10 @@ QPDFJob::handlePageSpecs(QPDF& pdf, std::vector>& page_hea auto n_collate = m->collate.size(); auto n_specs = parsed_specs.size(); + if (!(n_collate == 0 || n_collate == 1 || n_collate == n_specs)) { + usage("--pages: if --collate has more than one value, it must have one value per page " + "specification"); + } if (n_collate > 0 && n_specs > 1) { // Collate the pages by selecting one page from each spec in order. When a spec runs out of // pages, stop selecting from it. -- cgit v1.2.3-54-g00ecf