Skip to content

Commit

Permalink
[FIXED JENKINS-23568] Be sure to release WorkspaceList.Lease determin…
Browse files Browse the repository at this point in the history
…istically in a finally block.

(cherry picked from commit 47fa17b)

Conflicts:
	changelog.html
  • Loading branch information
jglick authored and olivergondza committed Jan 29, 2015
1 parent 3520042 commit 16ee133
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion core/src/main/java/hudson/model/AbstractProject.java
Expand Up @@ -1451,9 +1451,9 @@ private PollingResult pollWithWorkspace(TaskListener listener, SCM scm, R lb, @N
// OTOH, if a concurrent build is chosen, the user is willing to create a multiple workspace,
// so better throughput is achieved over time (modulo the initial cost of creating that many workspaces)
// by having multiple workspaces
WorkspaceList.Lease lease = l.acquire(ws, !concurrentBuild);
Node node = lb.getBuiltOn();
Launcher launcher = ws.createLauncher(listener).decorateByEnv(getEnvironment(node,listener));
WorkspaceList.Lease lease = l.acquire(ws, !concurrentBuild);
try {
listener.getLogger().println("Polling SCM changes on " + node.getSelfLabel().getName());
LOGGER.fine("Polling SCM changes of " + getName());
Expand Down

0 comments on commit 16ee133

Please sign in to comment.