Bug related to DefaultFileRegion offset position for EpollSocketChannel?

[Reynold Xin]

This is the latest writeFileRegion code on 4.0 branch:

    private boolean writeFileRegion(ChannelOutboundBuffer in, DefaultFileRegion region) throws Exception {

        if (region.transfered() >= region.count()) {

            in.remove();

            return true;

        }

        boolean done = false;

        long flushedAmount = 0;

        for (int i = config().getWriteSpinCount() - 1; i >= 0; i --) {

            long expected = region.count() - region.position();

            long localFlushedAmount = Native.sendfile(fd, region, region.transfered(), expected);

            if (localFlushedAmount == 0) {

                // Returned EAGAIN need to set EPOLLOUT

                setEpollOut();

                break;

            }

            flushedAmount += localFlushedAmount;

            if (region.transfered() >= region.count()) {

                done = true;

                break;

            }

        }

        in.progress(flushedAmount);

        if (done) {

            in.remove();

        }

        return done;

    }

Should the highlighted part be region.position() + region.transferred() ?

Java_io_netty_channel_epoll_Native_sendfile's 3rd argument is off (offset), and if I am reading this correctly, writeFileRegion here ignores the offset specified by DefaultFileRegion.

[이희승 (Trustin Lee)]

Thanks for reporting. It's fixed now: https://github.com/netty/netty/issues/2764

--

---
You received this message because you are subscribed to the Google Groups "Netty discussions" group.
To unsubscribe from this group and stop receiving emails from it, send an email to netty+un...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

--

https://twitter.com/trustin
https://twitter.com/trustin_ko
https://twitter.com/netty_project

[Norman Maurer]

Thanks for taking care :)