Merge pull request #232 from oleg-nenashev/findbugs-cleanup/JENKINS-3…

…7566-ProxyOutputStream-finalizer [JENKINS-37566] - Stop nullifying channel in ProxyOutputStream#finalize()
jenkinsci · Nov 21, 2017 · 385744f · 385744f
2 parents 2d5624c + d620065
commit 385744f
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/src/main/java/hudson/remoting/ProxyOutputStream.java b/src/main/java/hudson/remoting/ProxyOutputStream.java
@@ -181,9 +181,8 @@ protected void finalize() throws Throwable {
         super.finalize();
         // if we haven't done so, release the exported object on the remote side.
         // if the object is auto-unexported, the export entry could have already been removed.
-        if(channel!=null) {
+        if(channel != null && oid != -1) {
             channel.send(new Unexport(channel.newIoId(),oid));
-            channel = null;
             oid = -1;
         }
     }