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

Release the cumulation buffer after firing upstream so we don't end up

Browse Source 
with a "leak" because of a very big ChannelBuffer. This patch is
extracted of the pull request 39. Thanks to arya for the patch
This commit is contained in:
Norman Maurer 2011-11-03 11:25:24 +01:00
parent 56462ef91a
commit 31b319a5dc
1 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/main/java/org/jboss/netty/handler/codec/frame/FrameDecoder.java
View File

@ -298,6 +298,10 @@ public abstract class FrameDecoder extends SimpleChannelUpstreamHandler {
unfoldAndFireMessageReceived(context, remoteAddress, frame); unfoldAndFireMessageReceived(context, remoteAddress, frame);
} }
if (!cumulation.readable()) {
this.cumulation = null;
}
} }
private void unfoldAndFireMessageReceived(ChannelHandlerContext context, SocketAddress remoteAddress, Object result) { private void unfoldAndFireMessageReceived(ChannelHandlerContext context, SocketAddress remoteAddress, Object result) {