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/07/10 17:17:35 UTC
svn commit: r1801504 -
/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDColorSpace.java
Author: tilman
Date: Mon Jul 10 17:17:34 2017
New Revision: 1801504
URL: http://svn.apache.org/viewvc?rev=1801504&view=rev
Log:
PDFBOX-2852: avoid potential AIOOB / CC exceptions
Modified:
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDColorSpace.java
Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDColorSpace.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDColorSpace.java?rev=1801504&r1=1801503&r2=1801504&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDColorSpace.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDColorSpace.java Mon Jul 10 17:17:34 2017
@@ -153,7 +153,16 @@ public abstract class PDColorSpace imple
else if (colorSpace instanceof COSArray)
{
COSArray array = (COSArray)colorSpace;
- COSName name = (COSName)array.getObject(0);
+ if (array.size() == 0)
+ {
+ throw new IOException("Colorspace array is empty");
+ }
+ COSBase base = array.getObject(0);
+ if (!(base instanceof COSName))
+ {
+ throw new IOException("First element in colorspace array must be a name");
+ }
+ COSName name = (COSName) base;
// TODO cache these returned color spaces?