ResourceManager shouldn't set threadLocked to true if resource acquisition got interrupted.

--
MOS_MIGRATED_REVID=131703083

1 files changed, 7 insertions, 7 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/actions/ResourceManager.java b/src/main/java/com/google/devtools/build/lib/actions/ResourceManager.java
index 9d5edc3153..2f0483e7eb 100644
--- a/src/main/java/com/google/devtools/build/lib/actions/ResourceManager.java
+++ b/src/main/java/com/google/devtools/build/lib/actions/ResourceManager.java
@@ -227,14 +227,14 @@ public class ResourceManager {
         }
       }
       throw e;
-    } finally {
-      threadLocked.set(resources != ResourceSet.ZERO);
-      acquired(owner);
+    }
 
-      // Profile acquisition only if it waited for resource to become available.
-      if (latch != null) {
-        p.complete();
-      }
+    threadLocked.set(resources != ResourceSet.ZERO);
+    acquired(owner);
+
+    // Profile acquisition only if it waited for resource to become available.
+    if (latch != null) {
+      p.complete();
     }
 
     return new ResourceHandle(this, owner, resources);