ICEpdf
  1. ICEpdf
  2. PDF-337

Incorrect decode of jpeg image

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 4.2.2
          • Fix Version/s: 5.0.1
          • Component/s: Core/Parsing
          • Labels:
            None
          • Environment:
            any

            Description

            The PDF in question displays a black image on the page, obfuscating the black text that is drawn above it. In our rendering core the JPEG image is detected as grey encoded. My first guess is that this is incorrect.

            This issue is complicated by the the fact that our current JPEG support is handle by a now deprecated and removed library in JDK 1.7. As a result we'll have to look at this issue using an alternative image library.

              Activity

              Repository Revision Date User Message
              ICEsoft Public SVN Repository #34532 Thu Apr 25 10:41:19 MDT 2013 patrick.corless PDF-337 updated the jpeg decode to apply the colour space when handing jpegs with gray encoding.
              Files Changed
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/ImageUtility.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/Separation.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/ImageStream.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #34635 Wed May 01 13:37:28 MDT 2013 patrick.corless PDF-337 removed the namedColor logic to fix a regression in how we a apply Separation to jpeg images. Experimental.
              Files Changed
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/Separation.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #34680 Thu May 02 14:37:02 MDT 2013 patrick.corless PDF-337 added namd color logic back in but with the twist that we apply white for a tint value of 0.
              Files Changed
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/Separation.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #34701 Fri May 03 11:36:44 MDT 2013 patrick.corless PDF-337 reworked separation named colours gain, only apply the color name now on fill and stroke operations, images will always get the alternate colour space. Also updated the PColorSpace to add referenced colour modes to the library to avoid the reinitialization of the same object.
              Files Changed
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/PColorSpace.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/Lab.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/Indexed.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/DeviceCMYK.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/Separation.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/PatternColor.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/DeviceN.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/ImageStream.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/ICCBased.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/DeviceRGB.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/CalRGB.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/util/content/AbstractContentParser.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/DeviceGray.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #34707 Fri May 03 14:07:13 MDT 2013 patrick.corless PDF-337 reworked separation named colours gain, only apply the color name now on fill and stroke operations that we know we can paint in rgb. Also added basic CalGray colour space.
              Files Changed
              Commit graph ADD /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/CalGray.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/PColorSpace.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/Separation.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/CalRGB.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #34727 Mon May 06 10:46:21 MDT 2013 patrick.corless PDF-337 fixed a boolean logic typo, fixes the proper detection of black and blue named colours.
              Files Changed
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/Separation.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #34777 Tue May 07 13:10:34 MDT 2013 patrick.corless PDF-337 update separation colour to only apply tint on scn and SCN.
              Files Changed
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/util/content/OContentParser.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/Separation.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/util/content/AbstractContentParser.java

                People

                • Assignee:
                  Patrick Corless
                  Reporter:
                  Patrick Corless
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: