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 2021/04/30 06:07:11 UTC
svn commit: r1889317 -
/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/COSParser.java
Author: lehmi
Date: Fri Apr 30 06:07:11 2021
New Revision: 1889317
URL: http://svn.apache.org/viewvc?rev=1889317&view=rev
Log:
PDFBOX-4892: use current position for loop detection due to skipped spaces
Modified:
pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/COSParser.java
Modified: pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/COSParser.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/COSParser.java?rev=1889317&r1=1889316&r2=1889317&view=diff
==============================================================================
--- pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/COSParser.java (original)
+++ pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/COSParser.java Fri Apr 30 06:07:11 2021
@@ -318,9 +318,10 @@ public class COSParser extends BaseParse
{
// seek to xref table
source.seek(prev);
-
// skip white spaces
skipSpaces();
+ // save current position instead of prev due to skipped spaces
+ prevSet.add(source.getPosition());
// -- parse xref
if (source.peek() == X)
{
@@ -398,7 +399,6 @@ public class COSParser extends BaseParse
{
throw new IOException("/Prev loop at offset " + prev);
}
- prevSet.add(prev);
}
// ---- build valid xrefs out of the xref chain
xrefTrailerResolver.setStartxref(startXrefOffset);