Problem displaying CMYK jpeg images

Question

I have a problem with a CMYK jpeg image.
I know that about supported JPEGs in JRE - JCS_CMYK and JCS_YCCK are not supported directly.
And I found here how to load the image:
http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=4799903
http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=5100094
I used both the suggestions described to load it, but when the image is displayed the colors are wrong, are very different from the one displayed in PhotoShop, for example.
Any suggestion are very appreciated. If it's necessary i can send you the image.
Roberto
Here is the code
package com.cesarer.example.crop;
import java.awt.*;
import java.awt.event.*;
import java.awt.image.*;
import java.io.*;
import java.util.Iterator;
import javax.imageio.ImageIO;
import javax.imageio.ImageReader;
import javax.imageio.stream.ImageInputStream;
import javax.swing.*;
import javax.swing.event.MouseInputAdapter;
public class CroppingRaster extends JPanel
    BufferedImage image;
    Dimension size;
    Rectangle clip;
    boolean showClip;
    public CroppingRaster(BufferedImage image)
        this.image = image;
        size = new Dimension(image.getWidth(), image.getHeight());
        showClip = false;
    protected void paintComponent(Graphics g)
        super.paintComponent(g);
        Graphics2D g2 = (Graphics2D)g;
        g2.setRenderingHint(RenderingHints.KEY_ANTIALIASING,
                            RenderingHints.VALUE_ANTIALIAS_ON);
        int x = (getWidth() - size.width)/2;
        int y = (getHeight() - size.height)/2;
        g2.drawImage(image, x, y, this);
        if(showClip)
            if(clip == null)
                createClip();
            g2.setPaint(Color.red);
            g2.draw(clip);
    public void setClip(int x, int y)
        // keep clip within raster
        int x0 = (getWidth() - size.width)/2;
        int y0 = (getHeight() - size.height)/2;
        if(x < x0 || x + clip.width  > x0 + size.width ||
           y < y0 || y + clip.height > y0 + size.height)
            return;
        clip.setLocation(x, y);
        repaint();
    public Dimension getPreferredSize()
        return size;
    private void createClip()
        clip = new Rectangle(140, 140);
        clip.x = (getWidth() - clip.width)/2;
        clip.y = (getHeight() - clip.height)/2;
    private void clipImage()
        BufferedImage clipped = null;
        try
            int w = clip.width;
            int h = clip.height;
            int x0 = (getWidth()  - size.width)/2;
            int y0 = (getHeight() - size.height)/2;
            int x = clip.x - x0;
            int y = clip.y - y0;
            clipped = image.getSubimage(x, y, w, h);
        catch(RasterFormatException rfe)
            System.out.println("raster format error: " + rfe.getMessage());
            return;
        JLabel label = new JLabel(new ImageIcon(clipped));
        JOptionPane.showMessageDialog(this, label, "clipped image",
                                      JOptionPane.PLAIN_MESSAGE);
    private JPanel getUIPanel()
        final JCheckBox clipBox = new JCheckBox("show clip", showClip);
        clipBox.addActionListener(new ActionListener()
            public void actionPerformed(ActionEvent e)
                showClip = clipBox.isSelected();
                repaint();
        JButton clip = new JButton("clip image");
        clip.addActionListener(new ActionListener()
            public void actionPerformed(ActionEvent e)
                clipImage();
        JPanel panel = new JPanel();
        panel.add(clipBox);
        panel.add(clip);
        return panel;
    public static void main(String[] args) throws IOException
         File file = new File("src/ABWRACK_4C.jpg");
         // Find a JPEG reader which supports reading Rasters.
        Iterator readers = ImageIO.getImageReadersByFormatName("JPEG");
        ImageReader reader = null;
        while(readers.hasNext()) {
            reader = (ImageReader)readers.next();
            if(reader.canReadRaster()) {
                break;
     // Set the input.
        ImageInputStream input =
            ImageIO.createImageInputStream(file);
        reader.setInput(input);
        Raster raster = reader.readRaster(0, null);
         BufferedImage bi = new BufferedImage(raster.getWidth(),
                raster.getHeight(),
                BufferedImage.TYPE_4BYTE_ABGR);
         // Set the image data.
         bi.getRaster().setRect(raster);
        // Cropping test = new Cropping(ImageIO.read(file));
         CroppingRaster test = new CroppingRaster(bi);
        ClipMoverRaster mover = new ClipMoverRaster(test);
        test.addMouseListener(mover);
        test.addMouseMotionListener(mover);
        JFrame f = new JFrame();
        f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        f.getContentPane().add(new JScrollPane(test));
        f.getContentPane().add(test.getUIPanel(), "South");
        f.setSize(400,400);
        f.setLocation(200,200);
        f.setVisible(true);
class ClipMoverRaster extends MouseInputAdapter
    CroppingRaster cropping;
    Point offset;
    boolean dragging;
    public ClipMoverRaster(CroppingRaster c)
        cropping = c;
        offset = new Point();
        dragging = false;
    public void mousePressed(MouseEvent e)
        Point p = e.getPoint();
        if(cropping.clip.contains(p))
            offset.x = p.x - cropping.clip.x;
            offset.y = p.y - cropping.clip.y;
            dragging = true;
    public void mouseReleased(MouseEvent e)
        dragging = false;
    public void mouseDragged(MouseEvent e)
        if(dragging)
            int x = e.getX() - offset.x;
            int y = e.getY() - offset.y;
            cropping.setClip(x, y);
}

Answer 1

The loading process It takes about 10 seconds, and every repaint it takes the same time.The painting yes, the loading no. It shouldn't take much more time to load the image then an ordinary JPEG. In particular, your now using a "natively accelerated" JPEG image reader, so it should take less time on average to load any JPEG. 10 seconds is absurd. Are you sure your not including the drawing time in that figure?
- Another problem: JAI-ImageIO is not available for MAC OS X platform.This is somewhat true. You can take the imageio.jar file and package it together with your program (adding it to the class path). This will give you most of the functionality of JAI-ImageIO; in particular the TIFFImageReader. You just won't be able to use the two natively accelerated ImageReaders that comes with installing.
Right now, you're using the accelerated CLibJPEGImageReader to read the image. You can't use this image reader on the MAC, so you have to take the approach mentioned in the 2nd bug report you originally posted (i.e. use an arbitrary CMYK profile).
The conversion is too long more than 30 seconds.The code you posted could be simplified to
BufferedImage rgbImage = new BufferedImage(w,h,
            BufferedImage.3_BYTE_BGR);
ColorConvertOp op = new ColorConvertOp(null);
op.filter(cmykImage,rgbImage);But again, 30 seconds is an absurd value. After a little preparation, converting from one ICC_Profile to another is essentially a look up operation. What does,
System.out.println(cmykImage.getColorModel().getColorSpace());print out? If it's not an instance of ICC_ColorSpace, then I can understand the 30 seconds. If it is, then something is dreadfully wrong.
the RGB bufferedImage that i get after the transformation when it is displayed is much more brightThis is the "one last problem that pops up" that I hinted at. It's because some gamma correction is going on in the background. I have a solution, but it's programming to the implementation of the ColorConvertOp class and not the API. So I'm not sure about its portability between java versions or OS's.
import javax.swing.*;
import java.awt.color.ICC_ColorSpace;
import java.awt.color.ICC_Profile;
import java.awt.image.BufferedImage;
import java.awt.image.ColorConvertOp;
import javax.imageio.ImageIO;
import java.io.File;
public class CMYKTest {
     public static void main(String[] args) throws Exception{
        long beginStamp, endStamp;
        //load image
        beginStamp = System.currentTimeMillis();
        BufferedImage img = ImageIO.read(/*cmyk image file*/);
        endStamp = System.currentTimeMillis();
        System.out.println("Time to load image: " + (endStamp-beginStamp));
        //color convert
        BufferedImage dest = new BufferedImage(img.getWidth(),
                                               img.getHeight(),
                                               BufferedImage.TYPE_3BYTE_BGR);
        beginStamp = System.currentTimeMillis();
        sophisticatedColorConvert(img,dest);
        endStamp = System.currentTimeMillis();
        System.out.println("Time to color convert: " + (endStamp-beginStamp));
        //display
        JFrame frame = new JFrame();
        frame.setContentPane(new JScrollPane(new JLabel(new ImageIcon(dest))));
        frame.setSize(500,500);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setVisible(true);
    public static BufferedImage sophisticatedColorConvert(BufferedImage src,
                                                          BufferedImage dst) {
        ICC_ColorSpace srcCS =
                (ICC_ColorSpace) src.getColorModel().getColorSpace();
        ICC_Profile srcProf = srcCS.getProfile();
        byte[] header = srcProf.getData(ICC_Profile.icSigHead);
        intToBigEndian(ICC_Profile.icSigInputClass,header,12);
        srcProf.setData(ICC_Profile.icSigHead,header);
        ColorConvertOp op = new ColorConvertOp(null);
        return op.filter(src, dst);
    private static void intToBigEndian(int value, byte[] array, int index) {
            array[index]   = (byte) (value >> 24);
            array[index+1] = (byte) (value >> 16);
            array[index+2] = (byte) (value >>  8);
            array[index+3] = (byte) (value);
}