Merge pull request #1730 from stephenc/jenkins-28690

[FIXED JENKINS-28690] Deadlock in hudson.model.Executor

core/src/main/java/hudson/model/Executor.java

@@ -205,7 +205,14 @@ public class Executor extends Thread implements ModelObject {
     }
 
     public Result abortResult() {
-        lock.readLock().lock();
+        // this method is almost always called as a result of the current thread being interrupted
+        // as a result we need to clean the interrupt flag so that the lock's lock method doesn't
+        // get confused and think it was interrupted while awaiting the lock
+        Thread.interrupted(); 
+        // we need to use a write lock as we may be repeatedly interrupted while processing and
+        // we need the same lock as used in void interrupt(Result,boolean,CauseOfInterruption...)
+        // JENKINS-28690
+        lock.writeLock().lock();
         try {
             Result r = interruptStatus;
             if (r == null) r =
@@ -213,7 +220,7 @@ public class Executor extends Thread implements ModelObject {
 
             return r;
         } finally {
-            lock.readLock().unlock();
+            lock.writeLock().unlock();
         }
     }