[FIXED JENKINS-11592] NPE in subversion polling of Maven jobs

changelog.html

@@ -55,6 +55,9 @@ Upcoming changes</a>
 <!-- Record your changes in the trunk here. -->
 <div id="trunk" style="display:none"><!--=TRUNK-BEGIN=-->
 <ul class=image>
+  <li class=bug>
+    Fixed NPE in Subversion polling of Maven jobs.
+    (<a href="https://issues.jenkins-ci.org/browse/JENKINS-11592">issue 11592</a>)
   <li class=rfe>
     CLI jar now has the version number in the manifest as well as the "-version" option.
 </ul>

maven-plugin/src/main/java/hudson/maven/MavenModuleSetBuild.java

@@ -165,15 +165,18 @@ public class MavenModuleSetBuild extends AbstractMavenBuild<MavenModuleSet,Maven
 
         
         mvn = mvn.forEnvironment(envs);
-        Node node = Computer.currentComputer().getNode();
-        if (node == null) {
-            log.getLogger().println("WARNING: cannot get current node: "+Computer.currentComputer());
-            return envs;
+        
+        Computer computer = Computer.currentComputer();
+        if (computer != null) { // just in case were not in a build
+            Node node = computer.getNode();
+            if (node != null) {
+                mvn = mvn.forNode(node, log);
+                
+                envs.put("M2_HOME", mvn.getHome());
+                envs.put("PATH+MAVEN", mvn.getHome() + "/bin");
+            }
         }
-        mvn = mvn.forNode(node, log);
         
-        envs.put("M2_HOME", mvn.getHome());
-        envs.put("PATH+MAVEN", mvn.getHome() + "/bin");
         return envs;
     }