You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ti...@apache.org on 2014/09/12 15:18:13 UTC

svn commit: r1624537 - /pdfbox/branches/1.8/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/PDFObjectStreamParser.java

Author: tilman
Date: Fri Sep 12 13:18:12 2014
New Revision: 1624537

URL: http://svn.apache.org/r1624537
Log:
PDFBOX-2345: avoid exception if there are more objects than in /N value of /ObjStrm

Modified:
    pdfbox/branches/1.8/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/PDFObjectStreamParser.java

Modified: pdfbox/branches/1.8/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/PDFObjectStreamParser.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/1.8/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/PDFObjectStreamParser.java?rev=1624537&r1=1624536&r2=1624537&view=diff
==============================================================================
--- pdfbox/branches/1.8/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/PDFObjectStreamParser.java (original)
+++ pdfbox/branches/1.8/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/PDFObjectStreamParser.java Fri Sep 12 13:18:12 2014
@@ -107,6 +107,11 @@ public class PDFObjectStreamParser exten
             {
                 object = new COSObject(cosObject);
                 object.setGenerationNumber( COSInteger.ZERO );
+                if (objectCounter >= objectNumbers.size())
+                {
+                    LOG.error("/ObjStm (object stream) has more objects than /N " + numberOfObjects);
+                    break;
+                }
                 COSInteger objNum =
                     COSInteger.get( objectNumbers.get( objectCounter).intValue() );
                 object.setObjectNumber( objNum );