[JENKINS-2786] Makes no sense for Computer.currentComputer to return …

…MasterComputer when not on an executor thread.
jenkinsci · Nov 5, 2014 · 0b0b868 · 0b0b868
1 parent e607048
commit 0b0b868
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/core/src/main/java/hudson/model/Computer.java b/core/src/main/java/hudson/model/Computer.java
@@ -1313,11 +1313,11 @@ public void doProgressiveLog( StaplerRequest req, StaplerResponse rsp) throws IO
      * Gets the current {@link Computer} that the build is running.
      * This method only works when called during a build, such as by
      * {@link hudson.tasks.Publisher}, {@link hudson.tasks.BuildWrapper}, etc.
+     * @return the {@link Computer} associated with {@link Executor#currentExecutor}, or null if not on an executor thread
      */
-    public static Computer currentComputer() {
+    public static @Nullable Computer currentComputer() {
         Executor e = Executor.currentExecutor();
-        // If no executor then must be on master node
-        return e != null ? e.getOwner() : Jenkins.getInstance().toComputer();
+        return e != null ? e.getOwner() : null;
     }
 
     /**