andreacavalli/netty5
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Close ObjectInputStream in ObjectDecoder.decode(...)

Browse Source 
Motivation:

We create a new CompactObjectInputStream with ByteBufInputStream in ObjectDecoder.decode(...) method and don't close this InputStreams before return statement.

Modifications:

Save link to the ObjectInputStream and close it before return statement.

Result:

Close InputStreams and clean up unused resources. It will be better for GC.
This commit is contained in:
Idel Pivnitskiy 2014-07-20 00:02:03 +04:00 committed by Norman Maurer
parent 72173f0d16
commit f62777af11
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
codec/src/main/java/io/netty/handler/codec/serialization/ObjectDecoder.java
View File

@ -20,6 +20,7 @@ import io.netty.buffer.ByteBufInputStream;
import io.netty.channel.ChannelHandlerContext;
import io.netty.handler.codec.LengthFieldBasedFrameDecoder;
import java.io.ObjectInputStream;
import java.io.ObjectOutputStream;
import java.io.StreamCorruptedException;
@ -70,8 +71,10 @@ public class ObjectDecoder extends LengthFieldBasedFrameDecoder {
return null;
}
return new CompactObjectInputStream(
new ByteBufInputStream(frame), classResolver).readObject();
ObjectInputStream is = new CompactObjectInputStream(new ByteBufInputStream(frame), classResolver);
Object result = is.readObject();
is.close();
return result;
}
@Override