andreacavalli/netty-incubator-buffer-api
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Make MemSegBuffer.compact throw correct exception when read-only

Browse Source
This commit is contained in:
Chris Vest 2021-06-01 11:20:58 +02:00
parent 7f74dbc7aa
commit eaee4350b7
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
buffer-memseg/src/main/java/io/netty/buffer/api/memseg/MemSegBuffer.java
View File

@ -17,6 +17,7 @@ package io.netty.buffer.api.memseg;
import io.netty.buffer.ByteBuf; import io.netty.buffer.ByteBuf;
import io.netty.buffer.api.BufferAllocator; import io.netty.buffer.api.BufferAllocator;
import io.netty.buffer.api.BufferReadOnlyException;
import io.netty.buffer.api.adaptor.BufferIntegratable; import io.netty.buffer.api.adaptor.BufferIntegratable;
import io.netty.buffer.api.adaptor.ByteBufAdaptor; import io.netty.buffer.api.adaptor.ByteBufAdaptor;
import io.netty.buffer.api.adaptor.ByteBufAllocatorAdaptor; import io.netty.buffer.api.adaptor.ByteBufAllocatorAdaptor;
@ -618,7 +619,7 @@ class MemSegBuffer extends ResourceSupport<Buffer, MemSegBuffer> implements Buff
throw attachTrace(new IllegalStateException("Buffer must be owned in order to compact.")); throw attachTrace(new IllegalStateException("Buffer must be owned in order to compact."));
} }
if (readOnly()) { if (readOnly()) {
throw new IllegalStateException("Buffer must be writable in order to compact, but was read-only."); throw new BufferReadOnlyException("Buffer must be writable in order to compact, but was read-only.");
} }
int distance = roff; int distance = roff;
if (distance == 0) { if (distance == 0) {