From 718bf2fa459fd6f79a3c06d835faa180ef9604a1 Mon Sep 17 00:00:00 2001 From: Norman Maurer Date: Tue, 12 Apr 2016 16:27:02 +0200 Subject: [PATCH] Fix resource-leak which was reported as a result of commit 69070c37baf55e181f9270270f7cbf25958ba9b3 --- .../handler/codec/http/HttpResponseDecoderTest.java | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/codec-http/src/test/java/io/netty/handler/codec/http/HttpResponseDecoderTest.java b/codec-http/src/test/java/io/netty/handler/codec/http/HttpResponseDecoderTest.java index 82d8437009..0376258e98 100644 --- a/codec-http/src/test/java/io/netty/handler/codec/http/HttpResponseDecoderTest.java +++ b/codec-http/src/test/java/io/netty/handler/codec/http/HttpResponseDecoderTest.java @@ -15,6 +15,7 @@ */ package io.netty.handler.codec.http; +import io.netty.buffer.ByteBuf; import io.netty.buffer.Unpooled; import io.netty.channel.embedded.EmbeddedChannel; import io.netty.handler.codec.TooLongFrameException; @@ -576,7 +577,16 @@ public class HttpResponseDecoderTest { assertThat(ch.finish(), is(true)); - assertEquals(ch.readInbound(), Unpooled.wrappedBuffer(otherData)); + ByteBuf expected = Unpooled.wrappedBuffer(otherData); + ByteBuf buffer = ch.readInbound(); + try { + assertEquals(expected, buffer); + } finally { + expected.release(); + if (buffer != null) { + buffer.release(); + } + } } @Test