You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by se...@apache.org on 2007/10/26 08:39:56 UTC

svn commit: r588539 - /directory/studio/trunk/studio-ldapbrowser-core/src/main/java/org/apache/directory/studio/ldapbrowser/core/jobs/AbstractEclipseJob.java

Author: seelmann
Date: Thu Oct 25 23:39:53 2007
New Revision: 588539

URL: http://svn.apache.org/viewvc?rev=588539&view=rev
Log:
Fixed potential NPE when running jobs.

Modified:
    directory/studio/trunk/studio-ldapbrowser-core/src/main/java/org/apache/directory/studio/ldapbrowser/core/jobs/AbstractEclipseJob.java

Modified: directory/studio/trunk/studio-ldapbrowser-core/src/main/java/org/apache/directory/studio/ldapbrowser/core/jobs/AbstractEclipseJob.java
URL: http://svn.apache.org/viewvc/directory/studio/trunk/studio-ldapbrowser-core/src/main/java/org/apache/directory/studio/ldapbrowser/core/jobs/AbstractEclipseJob.java?rev=588539&r1=588538&r2=588539&view=diff
==============================================================================
--- directory/studio/trunk/studio-ldapbrowser-core/src/main/java/org/apache/directory/studio/ldapbrowser/core/jobs/AbstractEclipseJob.java (original)
+++ directory/studio/trunk/studio-ldapbrowser-core/src/main/java/org/apache/directory/studio/ldapbrowser/core/jobs/AbstractEclipseJob.java Thu Oct 25 23:39:53 2007
@@ -149,6 +149,18 @@
 
     public final void execute()
     {
+        // prevent from running the job if there are no connections 
+        Connection[] connections = getConnections();
+        for ( Connection connection : connections )
+        {
+            if(connection == null)
+            {
+                // TODO: logger
+                System.out.println( "Null connection: " + this.getClass().getName() );
+                return;
+            }
+        }
+        
         setUser( true );
         schedule();
     }