You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by le...@apache.org on 2023/06/24 14:27:16 UTC

svn commit: r1910589 - /pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/PDFXrefStreamParser.java

Author: lehmi
Date: Sat Jun 24 14:27:16 2023
New Revision: 1910589

URL: http://svn.apache.org/viewvc?rev=1910589&view=rev
Log:
PDFBOX-5623: don't rely on index entries being sorted ascending

Modified:
    pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/PDFXrefStreamParser.java

Modified: pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/PDFXrefStreamParser.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/PDFXrefStreamParser.java?rev=1910589&r1=1910588&r2=1910589&view=diff
==============================================================================
--- pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/PDFXrefStreamParser.java (original)
+++ pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/PDFXrefStreamParser.java Sat Jun 24 14:27:16 2023
@@ -198,11 +198,12 @@ public class PDFXrefStreamParser extends
                 }
                 long sizeValue = ((COSInteger) base).longValue();
                 start[counter] = startValue;
-                end[counter++] = startValue + sizeValue;
+                end[counter] = startValue + sizeValue;
+                maxValue = Math.max(maxValue, end[counter]);
+                counter++;
             }
             currentNumber = start[0];
             currentEnd = end[0];
-            maxValue = end[counter - 1];
         }
 
         @Override