Ignore null reporters field and avoid a NPE.

The NPE can occur if a build fails during checkout and hence the run phase never gets called. See JENKINS-10831 for an example of this happening. (cherry picked from commit 2fa4018)
jenkinsci · Feb 2, 2012 · d630404 · d630404
1 parent bafad70
commit d630404
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/maven-plugin/src/main/java/hudson/maven/MavenBuild.java b/maven-plugin/src/main/java/hudson/maven/MavenBuild.java
@@ -752,8 +752,11 @@ listener,new ProxyImpl(),
         }
 
         public void post2(BuildListener listener) throws Exception {
-            for (MavenReporter reporter : reporters)
-                reporter.end(MavenBuild.this,launcher,listener);
+            if (reporters != null) {
+                for (MavenReporter reporter : reporters) {
+                    reporter.end(MavenBuild.this,launcher,listener);
+                }
+            }
         }
 
     }