Merge pull request #1528 from m-holger/fuzz

Fix #1527

Merge pull request #1528 from m-holger/fuzz
Fix #1527
m-holger · GitHub
2 parents 800e89de 8abb42e3
Showing 5 changed files with 6 additions and 2 deletions
fuzz/CMakeLists.txt
fuzz/qpdf_extra/440599107.fuzz
fuzz/qpdf_extra/440747125.fuzz
fuzz/qtest/fuzz.test
libqpdf/NNTree.cc
@@ -159,6 +159,8 @@ set(CORPUS_OTHER
   409905355.fuzz
   411312393.fuzz
   433311400.fuzz
+  440599107.fuzz
+  440747125.fuzz
   4720043549327360.fuzz
   5109284021272576.fuzz
   6489005569146880.fuzz
@@ -11,7 +11,7 @@ my $td = new TestDriver(&#39;fuzz&#39;);
  
 my $qpdf_corpus = $ENV{'QPDF_FUZZ_CORPUS'} || die "must set QPDF_FUZZ_CORPUS";
  
-my $n_qpdf_files = 99;       # increment when adding new files
+my $n_qpdf_files = 101;       # increment when adding new files
  
 my @fuzzers = (
     ['ascii85' => 1],
@@ -103,10 +103,12 @@ NNTreeIterator::getNextKid(PathElement&amp; pe, bool backward)
     }
 }
  
+// iterator can be incremented or decremented, or dereferenced. This does not imply that it points
+// to a valid item.
 bool
 NNTreeIterator::valid() const
 {
-    return item_number >= 0 && ivalue.first && ivalue.second;
+    return item_number >= 0;
 }
  
 void