Solved

Security exception on ImageIcon.getImage()

Posted on 2007-04-03
6
675 Views
Last Modified: 2012-06-21
Hi,

I'm writing an applet which uses images downloaded from google images. Generally this works fine but in some cases when the image isn't available i get a security exception which is not caught by the following code:
BufferedImage bufferedImage = null;
    try{
            //convert string to url as Imageicon gets url address.
            URL url = new URL(imageLink);
            //ensures that all pixels have been loaded before returning
            Image img = new ImageIcon(url).getImage();
            bufferedImage = getBufferedImageFromImage(img);
       } catch(Exception e) {
            e.printStackTrace();
       }
       return bufferedImage;
      
The error stack generated is:
Uncaught error fetching image:
java.lang.SecurityException
      at java.lang.SecurityManager.checkPermission(Unknown Source)
      at java.lang.SecurityManager.checkConnect(Unknown Source)
      at sun.awt.image.URLImageSource.checkSecurity(Unknown Source)
      at sun.awt.image.ImageRepresentation.imageComplete(Unknown Source)
      at sun.awt.image.InputStreamImageSource.errorConsumer(Unknown Source)
at sun.awt.image.InputStreamImageSource.setDecoder(Unknown Source)
      at sun.awt.image.InputStreamImageSource.doFetch(Unknown Source)
      at sun.awt.image.ImageFetcher.fetchloop(Unknown Source)
      at sun.awt.image.ImageFetcher.run(Unknown Source)

Any comments would be appreciated.
Sophe
0
Comment
Question by:sophe
  • 4
  • 2
6 Comments
 
LVL 24

Expert Comment

by:sciuriware
ID: 18844877
Do you have access to that image file?
Generally you only have access to the server.

;JOOP!
0
 

Author Comment

by:sophe
ID: 18844973
sorry i don't understand your question.
it is simply using a url to a file found via google images.
here is an example url that causes this problem: http://6a.typepad.com/images/missing.gif 

sophe
0
 
LVL 24

Expert Comment

by:sciuriware
ID: 18848670
Copy that image to your project and access it as local resource.
In general a random file is not accessible by a JAVA client because of security restrictions.
You can't access files local to the client without access being granted, the same for
files on some random system, otherwise the former situation could be 'circumvented'.
So, copy the image to the directory of one of your class files and load it as in this snippet:

   /**
    * Safely get the path to a resource file via Class info.
    * <p>If the file is missing, assuming that the resource was an image file,<BR>
    * the packaged <b>MISSING.gif</b> icon is substituted,
    * using the class info <b>packageClass</b> retrieved elsewhere.</p>
    * @param  classinfo   a Class loaded from the root of relative pathname.
    * @param  filename    the relative pathname to the resource file.
    * @return             an URL to the resulting resource.
    */
   private static URL findResource(Class classInfo, String filename)
   {
   URL resourceURL;

      if(classInfo == null)
      {
         fatal("BAD CLASS INFO");
      }

      if((resourceURL = classInfo.getResource(filename)) == null)
      {
         error("RESOURCE FILE '" + filename + "' MISSING.");
         if((resourceURL
            = packageClass.getResource("res/MISSING.gif")) == null)
         {
            fatal("SUBSTITUTE IMAGE 'MISSING' NOT FOUND.");
         }
      }
      return(resourceURL);
   }


// And deploying this in my program elsewhere (assume JFrame mainFrame;   ):

   /**
    * Set the default icon on the program main JFrame.
    * <p>Can be used to reset the icon after calls with an argument.<BR>
    * The default icon should be <b>res/<program>.gif</b>.</p>
    */
   public static void setMainFrameIcon( )
   {
      mainFrame.setIconImage
      (
         Toolkit.getDefaultToolkit().getImage(findResource(this.getClass( ), "res/Application.gif"))
      );
   }

;JOOP!
0
Best Practices: Disaster Recovery Testing

Besides backup, any IT division should have a disaster recovery plan. You will find a few tips below relating to the development of such a plan and to what issues one should pay special attention in the course of backup planning.

 

Author Comment

by:sophe
ID: 18849340
The url is found dynamically by searching google images for a keyword entered by the user. This is done with the following code:
/**
*@param keyword is the object  to be searched in google
*/
public String makeURLCon (String keyword) {
    String source= null;
    try{
         //search Google images for the keyword
         URL url = new    
                URL("http://images.google.co.uk/images?svnum=10&hl=en&q="+keyword+"&btnG=Search");
                  
            URLConnection connection = url.openConnection();
            connection.setDoInput(true);
            //Sets the general request property
            connection.setRequestProperty("User-agent","Mozilla/4.0");
            InputStream inputStrm = connection.getInputStream();
            source = readSource(inputStrm);
                  
      }catch (Exception e) {
            e.printStackTrace();
      }
      return source;
}

Generally if the applet sees the image can't be downloaded it moves on to the next url in the source. But this doesn't happen when a SecurityException is thrown for some reason.

So this means that i can't save each image manually.
0
 
LVL 24

Expert Comment

by:sciuriware
ID: 18849627
If you are sure about that, simply catch it and go on searching:

    try
    {
// convert string to url as Imageicon gets url address.
            URL url = new URL(imageLink);
// ensures that all pixels have been loaded before returning
            Image img = new ImageIcon(url).getImage();
            bufferedImage = getBufferedImageFromImage(img);
    }
    catch(Exception e) { /* IGNORE EXCEPTION */ }

    return bufferedImage;

// See the corrected code, and heed my style.

;JOOP!
0
 
LVL 24

Accepted Solution

by:
sciuriware earned 250 total points
ID: 18849637
Additional remark:  the code that calls this must check that if null is returned
the decision must be made to retry or not!

And, on second thoughts,  it is better to replace the line:
                  catch(Exception e) { /* IGNORE EXCEPTION */ }
by:
                  catch(SecurityException e) { /* IGNORE EXCEPTION */ }

;JOOP!
0

Featured Post

Best Practices: Disaster Recovery Testing

Besides backup, any IT division should have a disaster recovery plan. You will find a few tips below relating to the development of such a plan and to what issues one should pay special attention in the course of backup planning.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Suggested Solutions

Java contains several comparison operators (e.g., <, <=, >, >=, ==, !=) that allow you to compare primitive values. However, these operators cannot be used to compare the contents of objects. Interface Comparable is used to allow objects of a cl…
In this post we will learn how to connect and configure Android Device (Smartphone etc.) with Android Studio. After that we will run a simple Hello World Program.
Viewers learn about the third conditional statement “else if” and use it in an example program. Then additional information about conditional statements is provided, covering the topic thoroughly. Viewers learn about the third conditional statement …
This tutorial covers a practical example of lazy loading technique and early loading technique in a Singleton Design Pattern.

773 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question