[FIXED JENKINS-4220] fixed a dead lock.

While the original report is in https://cloudbees.zendesk.com/tickets/427, this fix also takes care of JENKINS-4220. Originally-Committed-As: 53a31c31530166fc7a53900b58127c002a7e8cda
jenkinsci · Mar 1, 2011 · a901e3d · a901e3d
1 parent 36c11f5
commit a901e3d
Showing 1 changed file with 12 additions and 2 deletions.
diff --git a/src/main/java/hudson/maven/MavenModuleSetBuild.java b/src/main/java/hudson/maven/MavenModuleSetBuild.java
@@ -126,6 +126,8 @@ public class MavenModuleSetBuild extends AbstractMavenBuild<MavenModuleSet,Maven
 
     private String mavenVersionUsed;
 
+    private transient Object notifyModuleBuildLock = new Object();
+
     public MavenModuleSetBuild(MavenModuleSet job) throws IOException {
         super(job);
     }
@@ -134,6 +136,12 @@ public MavenModuleSetBuild(MavenModuleSet project, File buildDir) throws IOExcep
         super(project, buildDir);
     }
 
+    @Override
+    protected void onLoad() {
+        super.onLoad();
+        notifyModuleBuildLock = new Object();
+    }
+
     /**
      * Exposes {@code MAVEN_OPTS} to forked processes.
      *
@@ -457,7 +465,9 @@ public Fingerprint.RangeSet getDownstreamRelationship(AbstractProject that) {
 
             // actions need to be replaced atomically especially
             // given that two builds might complete simultaneously.
-            synchronized(this) {
+            // use a separate lock object since this synchronized block calls into plugins,
+            // which in turn can access other MavenModuleSetBuild instances, which will result in a dead lock.
+            synchronized(notifyModuleBuildLock) {
                 boolean modified = false;
 
                 List<Action> actions = getActions();
@@ -1318,7 +1328,7 @@ private String getRootPath(String prefix) {
      * Extra verbose debug switch.
      */
     public static boolean debug = Boolean.getBoolean( "hudson.maven.debug" );
-    
+
     @Override
     public MavenModuleSet getParent() {// don't know why, but javac wants this
         return super.getParent();