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

Correctly respect readerIndex of buffer when dumping.

Browse Source 
Motivation:

The current dumping code does not respect the readerIndex and so logs incorrect.

Modifications:

Respect readerIndex of ByteBuf

Result:

Correctly log content of buffer.
This commit is contained in:
Norman Maurer 2015-06-03 22:29:28 +02:00
parent a485ae68dc
commit cf54c04241
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
handler/src/main/java/io/netty/handler/logging/LoggingHandler.java
View File

@ -419,7 +419,7 @@ public class LoggingHandler extends ChannelDuplexHandler {
// Dump the rows which have 16 bytes. // Dump the rows which have 16 bytes.
for (int row = 0; row < fullRows; row ++) { for (int row = 0; row < fullRows; row ++) {
int rowStartIndex = row << 4; int rowStartIndex = (row << 4) + startIndex;
// Per-row prefix. // Per-row prefix.
appendHexDumpRowPrefix(dump, row, rowStartIndex); appendHexDumpRowPrefix(dump, row, rowStartIndex);
@ -440,7 +440,7 @@ public class LoggingHandler extends ChannelDuplexHandler {
// Dump the last row which has less than 16 bytes. // Dump the last row which has less than 16 bytes.
if (remainder != 0) { if (remainder != 0) {
int rowStartIndex = fullRows << 4; int rowStartIndex = (fullRows << 4) + startIndex;
appendHexDumpRowPrefix(dump, fullRows, rowStartIndex); appendHexDumpRowPrefix(dump, fullRows, rowStartIndex);
// Hex dump // Hex dump