You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ja...@apache.org on 2014/02/20 23:50:17 UTC

svn commit: r1570380 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDICCBased.java

Author: jahewson
Date: Thu Feb 20 22:50:17 2014
New Revision: 1570380

URL: http://svn.apache.org/r1570380
Log:
Fixed issue with invalid ICC profiles introuced in PDFBOX-1893

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDICCBased.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDICCBased.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDICCBased.java?rev=1570380&r1=1570379&r2=1570380&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDICCBased.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDICCBased.java Thu Feb 20 22:50:17 2014
@@ -124,7 +124,7 @@ public final class PDICCBased extends PD
             initialColor = new PDColor(initial);
 
             // create a color in order to trigger a ProfileDataException
-            // or CMMException due to invalid profiles, see PDFBOX-1295
+            // or CMMException due to invalid profiles, see PDFBOX-1295 and PDFBOX-1740
             new Color(awtColorSpace, new float[getNumberOfComponents()], 1f);
         }
         catch (RuntimeException e)
@@ -132,7 +132,8 @@ public final class PDICCBased extends PD
             if (e instanceof ProfileDataException || e instanceof CMMException)
             {
                 // fall back to alternateColorSpace color space
-                LOG.debug("Can't read ICC-profile, using alternate color space");
+                LOG.error("Can't read embedded ICC profile, using alternate color space");
+                awtColorSpace = null;
                 alternateColorSpace = getAlternateColorSpaces().get(0);
                 initialColor = alternateColorSpace.getInitialColor();
             }