[
https://issues.apache.org/jira/browse/ZOOKEEPER-1843?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13982916#comment-13982916
]
Hudson commented on ZOOKEEPER-1843:
-----------------------------------
FAILURE: Integrated in ZooKeeper-trunk #2304 (See
[https://builds.apache.org/job/ZooKeeper-trunk/2304/])
> Oddity in ByteBufferInputStream skip
> ------------------------------------
>
> Key: ZOOKEEPER-1843
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1843
> Project: ZooKeeper
> Issue Type: Bug
> Reporter: Justin SB
> Assignee: Bill Havanki
> Priority: Minor
> Fix For: 3.5.0
>
> Attachments: ZOOKEEPER-1843.patch, ZOOKEEPER-1843.v2.patch
>
>
> I was reading ByteBufferInputStream.java; here is the skip method:
> public long skip(long n) throws IOException {
> long newPos = bb.position() + n;
> if (newPos > bb.remaining()) {
> n = bb.remaining();
> }
> bb.position(bb.position() + (int) n);
> return n;
> }
> The first two lines look wrong; we compare a "point" (position) to a
> "distance" (remaining). I think the test should be if (newPos >= bb.limit()).
> Or more simply:
> public long skip(long n) throws IOException {
> int remaining = buffer.remaining();
> if (n > remaining) {
> n = remaining;
> }
> buffer.position(buffer.position() + (int) n);
> return n;
> }
--
This message was sent by Atlassian JIRA
(v6.2#6252)
