Want to protect your cyber security and still get fast solutions? Ask a secure question today.Go Premium

x
  • Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 1779
  • Last Modified:

JAI: Reading bad JPEGs

Here is the code I use:
    public int getImage(String fileName) {
        try{
            if (!fileName.toUpperCase().endsWith(".JPG") && !fileName.toUpperCase().endsWith(".JPEG")){
                return 0;
            }
          RenderedOp image = JAI.create("fileload", fileName);
            try{
                System.out.println(image.getParameters());
                System.out.println(image.getBounds());
            }catch (Exception e){
                return 0;
            }
            return 1;
        }catch (Exception e){
            e.printStackTrace();
            return 0;
        }
    }

All is fine, the create operation is ok. But when reading bad JPEGs (0 bytes ones or things that are not JPEGS at all) , the invoking of the methods on the image object is throwing an Exception (getParameters() seems to be the only one working)
All the others throw:
Error: IOException occurs when search for propriate codecs.

Occurs in: com.sun.media.jai.codec.ImageCodec

java.io.EOFException
      at com.sun.media.jai.codec.SeekableStream.readFully(SeekableStream.java:329)
      at com.sun.media.jai.codec.SeekableStream.readFully(SeekableStream.java:306)
      at com.sun.media.jai.codec.ImageCodec.getDecoderNames(ImageCodec.java:274)
      at com.sun.media.jai.opimage.StreamRIF.create(StreamRIF.java:80)
      at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
      at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
      at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
      at java.lang.reflect.Method.invoke(Method.java:324)
      at javax.media.jai.FactoryCache.invoke(FactoryCache.java:130)
      at javax.media.jai.OperationRegistry.invokeFactory(OperationRegistry.java:1682)
      at javax.media.jai.ThreadSafeOperationRegistry.invokeFactory(ThreadSafeOperationRegistry.java:481)
      at javax.media.jai.registry.RIFRegistry.create(RIFRegistry.java:340)
      at com.sun.media.jai.opimage.FileLoadRIF.create(FileLoadRIF.java:122)
      at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
      at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
      at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
      at java.lang.reflect.Method.invoke(Method.java:324)
      at javax.media.jai.FactoryCache.invoke(FactoryCache.java:130)
      at javax.media.jai.OperationRegistry.invokeFactory(OperationRegistry.java:1682)
      at javax.media.jai.ThreadSafeOperationRegistry.invokeFactory(ThreadSafeOperationRegistry.java:481)
      at javax.media.jai.registry.RIFRegistry.create(RIFRegistry.java:340)
      at javax.media.jai.RenderedOp.createInstance(RenderedOp.java:830)
      at javax.media.jai.RenderedOp.createRendering(RenderedOp.java:878)
      at javax.media.jai.RenderedOp.getColorModel(RenderedOp.java:2253)
      at myclass.getImage(ReadCMYKJPG.java:131)
      at myclass.main(ReadCMYKJPG.java:151)
And this happens a few times in a row.

Line 131 is                
System.out.println(image.getBounds());

Any idea how to check the RenderedOp so that I do not call the methids if it is not ok?

Thanks in advance
Venabili
0
Venabili
Asked:
Venabili
  • 4
  • 3
1 Solution
 
CEHJCommented:
>>(0 bytes ones or things that are not JPEGS at all)

Try this:

      public static boolean isJPEG(File jpgFile) {
            long length = jpgFile.length();
            if (length < 1) {
                  return false;
            }
            FileInputStream in = null;
            final int HEAD_LENGTH = 10;
            byte[] header = new byte[HEAD_LENGTH];
            try {
                  in = new FileInputStream(jpgFile);
                  in.read(header);
                  return "JFIF".equals(new String(header, 6, 4));
            }
            catch (IOException e) {
                  e.printStackTrace();
                  return false;
            }
            finally {
                  try {
                        if (in != null) {
                              in.close();
                        }
                  }
                  catch(IOException e) {
                        /* ignore */
                  }
            }
      }
0
 
VenabiliAuthor Commented:
It does not recognise a valid CMYK JPEG.
The part for the empty file is ok but the second part does not recognise the JPEG :)

Venabili
0
 
CEHJCommented:
Can you post a link to the image?
0
Independent Software Vendors: We Want Your Opinion

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

 
VenabiliAuthor Commented:
It's a little copyrighted and just now I have no software to produce another:(
But I think I solved it - once I handle correctly the 0 bytes ones, the JAI takes care for the others :)
So this works:

    public static boolean isJPEG(File jpgFile) {
        long length = jpgFile.length();
        if (length < 1) {
            return false;
        }
        return true;
    }
    public static BufferedImage getImage(String fileName) {
        try{
            if (!fileName.toUpperCase().endsWith(".JPG") && !fileName.toUpperCase().endsWith(".JPEG")){
                return null;
            }
            RenderedOp image = JAI.create("fileload", fileName);
            BufferedImage result = null;
            try {
                if (isJPEG(new File(fileName))) {
                    if (image.getColorModel().getNumComponents() == 4) {
                        result = convertCMYKtoRGB(image);
                    }
                } else {
                    return null;
                }
            } catch (Exception e) {
                return null;
            }
            return result;
        }catch (Exception e){
            return null;
        }
    }

I need sleep :( and you just earned easy points:) Thanks for the help :)
0
 
CEHJCommented:
:-) Make sure it still works when you rename X.gif to X.jpg ;-)
0
 
VenabiliAuthor Commented:
Seems to be ok :)
0
 
CEHJCommented:
Well you might just get back a BufferedImage containing a gif without complaint ;-)
0

Featured Post

Hire Technology Freelancers with Gigs

Work with freelancers specializing in everything from database administration to programming, who have proven themselves as experts in their field. Hire the best, collaborate easily, pay securely, and get projects done right.

  • 4
  • 3
Tackle projects and never again get stuck behind a technical roadblock.
Join Now