Ignore additional PDF417 barcode rows

Some barcode generators (e.g. Okapi) seem to generate additional barcode
rows at the end of the barcode, which can be decoded as valid barcode
rows, but actually contain garbage. The barcode metadata states that
these additional rows are not part of the barcode. So we ignore those
addional rows when parsing the barcode. This could potentially also
happen (though quite unlikely) if data after the barcode looks like a
barcode row.

core/src/main/java/com/google/zxing/pdf417/decoder/DetectionResultRowIndicatorColumn.java

@@ -122,7 +122,8 @@ int[] getRowHeights() throws FormatException {
       if (codeword != null) {
         int rowNumber = codeword.getRowNumber();
         if (rowNumber >= result.length) {
-          throw FormatException.getFormatInstance();
+          // We have more rows than the barcode metadata allows for, ignore them.
+          continue;
         }
         result[rowNumber]++;
       } // else throw exception?

core/src/main/java/com/google/zxing/pdf417/decoder/PDF417ScanningDecoder.java

@@ -343,7 +343,8 @@ private static BarcodeValue[][] createBarcodeMatrix(DetectionResult detectionRes
             int rowNumber = codeword.getRowNumber();
             if (rowNumber >= 0) {
               if (rowNumber >= barcodeMatrix.length) {
-                throw FormatException.getFormatInstance();
+                // We have more rows than the barcode metadata allows for, ignore them.
+                continue;
               }
               barcodeMatrix[rowNumber][column].setValue(codeword.getValue());
             }

core/src/test/java/com/google/zxing/pdf417/PDF417BlackBox1TestCase.java

@@ -29,8 +29,8 @@ public final class PDF417BlackBox1TestCase extends AbstractBlackBoxTestCase {
 
   public PDF417BlackBox1TestCase() {
     super("src/test/resources/blackbox/pdf417-1", new MultiFormatReader(), BarcodeFormat.PDF_417);
-    addTest(9, 9, 0.0f);
-    addTest(9, 9, 180.0f);
+    addTest(10, 10, 0.0f);
+    addTest(10, 10, 180.0f);
   }
 
 }

core/src/test/resources/blackbox/pdf417-1/11.txt

@@ -0,0 +1 @@
+This is just a test.