You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by ka...@apache.org on 2010/03/05 14:01:00 UTC
svn commit: r919399 -
/directory/apacheds/trunk/test-framework/src/main/java/org/apache/directory/server/core/integ/FrameworkRunner.java
Author: kayyagari
Date: Fri Mar 5 13:00:59 2010
New Revision: 919399
URL: http://svn.apache.org/viewvc?rev=919399&view=rev
Log:
fixed a NPE which can happen if the suite is not null and no suite level LdapServer was defined
Modified:
directory/apacheds/trunk/test-framework/src/main/java/org/apache/directory/server/core/integ/FrameworkRunner.java
Modified: directory/apacheds/trunk/test-framework/src/main/java/org/apache/directory/server/core/integ/FrameworkRunner.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/test-framework/src/main/java/org/apache/directory/server/core/integ/FrameworkRunner.java?rev=919399&r1=919398&r2=919399&view=diff
==============================================================================
--- directory/apacheds/trunk/test-framework/src/main/java/org/apache/directory/server/core/integ/FrameworkRunner.java (original)
+++ directory/apacheds/trunk/test-framework/src/main/java/org/apache/directory/server/core/integ/FrameworkRunner.java Fri Mar 5 13:00:59 2010
@@ -188,11 +188,14 @@
{
LdapServer suiteServer = suite.getLdapServer();
- for ( Transport transport : suiteServer.getTransports() )
+ if ( suiteServer != null )
{
- if ( minPort <= transport.getPort() )
+ for ( Transport transport : suiteServer.getTransports() )
{
- minPort = transport.getPort();
+ if ( minPort <= transport.getPort() )
+ {
+ minPort = transport.getPort();
+ }
}
}
}