Skip to content
Permalink
Browse files
[JENKINS-20769]
Unexport is new, so for older versions fall back to EOF.
Better to have a bogus error message than memory leak.
  • Loading branch information
kohsuke committed Mar 27, 2014
1 parent 9b24114 commit 57c3f1ab9e866ae237a568bd57382b6fe900ab98
Showing with 4 additions and 1 deletion.
  1. +4 −1 src/main/java/hudson/remoting/ProxyWriter.java
@@ -193,7 +193,10 @@ protected void finalize() throws Throwable {
// 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) {
channel.send(new Unexport(channel.newIoId(),oid));
if (channel.remoteCapability.supportsProxyWriter2_35())
channel.send(new Unexport(channel.newIoId(),oid));
else
channel.send(new EOF(channel.newIoId(),oid));
channel = null;
oid = -1;
}

0 comments on commit 57c3f1a

Please sign in to comment.