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/10/16 21:47:30 UTC

svn commit: r1632427 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/PDFParser.java

Author: tilman
Date: Thu Oct 16 19:47:30 2014
New Revision: 1632427

URL: http://svn.apache.org/r1632427
Log:
PDFBOX-2434: ignore /Version number that isn't a Name and put out warning

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/PDFParser.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/PDFParser.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/PDFParser.java?rev=1632427&r1=1632426&r2=1632427&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/PDFParser.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/PDFParser.java Thu Oct 16 19:47:30 2014
@@ -915,15 +915,20 @@ public class PDFParser extends BaseParse
         COSObject root = (COSObject) parsedTrailer.getItem(COSName.ROOT);
         if (root != null)
         {
-            COSName version =  (COSName) root.getItem(COSName.VERSION);
-            if (version != null)
+            COSBase item = root.getItem(COSName.VERSION);
+            if (item instanceof COSName)
             {
+                COSName version = (COSName) item;
                 float trailerVersion = Float.valueOf(version.getName());
                 if (trailerVersion > document.getVersion())
                 {
                     document.setVersion(trailerVersion);
                 }
             }
+            else if (item != null)
+            {
+                LOG.warn("Incorrect /Version entry is ignored: " + item);
+            }
         }
     }