Stefan Hajnoczi <stefa...@redhat.com> writes:

> On Thu, Jun 05, 2014 at 02:15:36PM +0200, Markus Armbruster wrote:
>> +    if (sector > total_sectors || nb_sectors > total_sectors - sector) {
>> +        return false;
>> +    }
>
> if (sector >= total_sectors || ...) {
I suspect reading bdrv_check_byte_request() put the '>' in my brain:

    if ((offset > len) || (len - offset < size))
        return -EIO;

Don't we need offset >= len here?

Reply via email to