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.
jenkinsci · Aug 26, 2011 · 2fa4018 · 2fa4018
1 parent 8dfa5e1
commit 2fa4018
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
@@ -750,8 +750,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);
+                }
+            }
         }
 
     }