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 2017/08/15 16:37:00 UTC

svn commit: r1805096 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/COSParser.java

Author: tilman
Date: Tue Aug 15 16:36:59 2017
New Revision: 1805096

URL: http://svn.apache.org/viewvc?rev=1805096&view=rev
Log:
PDFBOX-3896: throw IOException instead of UnsupportedOperationException

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

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/COSParser.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/COSParser.java?rev=1805096&r1=1805095&r2=1805096&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/COSParser.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/COSParser.java Tue Aug 15 16:36:59 2017
@@ -667,6 +667,12 @@ public class COSParser extends BaseParse
                                     stmObjects = new ArrayList<>();
                                     objToBeParsed.put(fileOffset, stmObjects);
                                 }
+                                // java does not have a test for immutable
+                                else if (!(stmObjects instanceof ArrayList))
+                                {
+                                    throw new IOException(obj + " cannot be assigned to offset " +
+                                            fileOffset + ", this belongs to " + stmObjects.get(0));
+                                }
                                 stmObjects.add(obj);
                             }
                         }