You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@continuum.apache.org by ct...@apache.org on 2011/09/01 13:40:16 UTC

svn commit: r1164024 - /continuum/trunk/continuum-xmlrpc/continuum-xmlrpc-server/src/main/java/org/apache/maven/continuum/xmlrpc/server/ContinuumServiceImpl.java

Author: ctan
Date: Thu Sep  1 11:40:16 2011
New Revision: 1164024

URL: http://svn.apache.org/viewvc?rev=1164024&view=rev
Log:
[CONTINUUM-2666] added a check to prevent NPE

Modified:
    continuum/trunk/continuum-xmlrpc/continuum-xmlrpc-server/src/main/java/org/apache/maven/continuum/xmlrpc/server/ContinuumServiceImpl.java

Modified: continuum/trunk/continuum-xmlrpc/continuum-xmlrpc-server/src/main/java/org/apache/maven/continuum/xmlrpc/server/ContinuumServiceImpl.java
URL: http://svn.apache.org/viewvc/continuum/trunk/continuum-xmlrpc/continuum-xmlrpc-server/src/main/java/org/apache/maven/continuum/xmlrpc/server/ContinuumServiceImpl.java?rev=1164024&r1=1164023&r2=1164024&view=diff
==============================================================================
--- continuum/trunk/continuum-xmlrpc/continuum-xmlrpc-server/src/main/java/org/apache/maven/continuum/xmlrpc/server/ContinuumServiceImpl.java (original)
+++ continuum/trunk/continuum-xmlrpc/continuum-xmlrpc-server/src/main/java/org/apache/maven/continuum/xmlrpc/server/ContinuumServiceImpl.java Thu Sep  1 11:40:16 2011
@@ -2221,7 +2221,10 @@ public class ContinuumServiceImpl
         ConfigurationService configurationService = continuum.getConfiguration();
         org.apache.continuum.configuration.BuildAgentGroupConfiguration buildAgentGroupConfiguration =
             configurationService.getBuildAgentGroup( name );
-        configurationService.removeBuildAgentGroup( buildAgentGroupConfiguration );
+        if ( buildAgentGroupConfiguration != null )
+        {
+            configurationService.removeBuildAgentGroup( buildAgentGroupConfiguration );
+        }
 
         return 0;
     }